-
Notifications
You must be signed in to change notification settings - Fork 28
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
feat(fight): Add trap effect (#27) #287
feat(fight): Add trap effect (#27) #287
Conversation
@@ -72,11 +77,12 @@ public final Castable action() { | |||
} | |||
|
|||
@Override | |||
public final Optional<Spell> spell() { | |||
@Pure | |||
public final @Nullable Spell spell() { |
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.
:(
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.
Je ne suis pas fan de l'API optional, et checkerframework apporte une bonne sécurité compile time, ce que je préfère. Mais bon l'idéal reste une vraie gestion native un peu comme en Kotlin, même si ça n'arrivera jamais en java 😢
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #287 +/- ##
==========================================
Coverage 96.08% 96.09%
- Complexity 8337 8414 +77
==========================================
Files 1322 1329 +7
Lines 21159 21413 +254
Branches 1265 1286 +21
==========================================
+ Hits 20330 20576 +246
- Misses 635 640 +5
- Partials 194 197 +3
☔ View full report in Codecov by Sentry. |
feat(fight): Add trap effect (Arakne#27)
No description provided.