-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
@External
annotation
#1269
@External
annotation
#1269
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1269 +/- ##
============================================
+ Coverage 91.04% 91.08% +0.03%
- Complexity 4667 4672 +5
============================================
Files 597 597
Lines 14801 14810 +9
Branches 842 845 +3
============================================
+ Hits 13476 13490 +14
+ Misses 1063 1061 -2
+ Partials 262 259 -3 |
@armiol, PTAL. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dmdashenkov LGTM. Please see my comment though.
import static com.google.common.base.Preconditions.checkNotNull; | ||
import static com.google.common.truth.Truth.assertThat; | ||
|
||
@DisplayName("`External` attribute can be declared") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this about the attribute or the annotation? Please make it more clear.
This PR introduces the
@External
annotation to replace the(external = true)
attribute in@Subscribe
,@React
, and@Command
annotations.