-
Notifications
You must be signed in to change notification settings - Fork 293
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
Issue 6303: RFE - Hazardous Liquid Pool #6476
base: master
Are you sure you want to change the base?
Issue 6303: RFE - Hazardous Liquid Pool #6476
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6476 +/- ##
============================================
- Coverage 28.57% 28.57% -0.01%
- Complexity 14407 14421 +14
============================================
Files 2798 2800 +2
Lines 275146 275291 +145
Branches 48678 48703 +25
============================================
+ Hits 78636 78671 +35
- Misses 191838 191942 +104
- Partials 4672 4678 +6 ☔ View full report in Codecov by Sentry. |
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.
Overall looks good.
Just saw this was still draft. Sorry. |
No worries! At least one of those things (the formatting on the big If) wasn't going to change - I'd copied it from another (old) method so I think I'll clean it up, extract it into its own method, and use it in both places. I'll request a review from you when I'm done with it all :) |
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.
Overall looks sane.
…Hazardous Liquid & Princess Support
Hey @rjhancock this is ready for review. |
I can take a look now, but if this is for next dev cycle maybe we should wait until after 0.50.03 is built tomorrow? |
This should wait for the next cycle, yeah, so no rush. Thank you! |
|| ((en.getMovementMode() == EntityMovementMode.WIGE) | ||
&& (en.getOriginalWalkMP() > 0) && !eruption)) | ||
&& !en.isImmobile()) { | ||
if (!isUnitEffectedByHazardousGround(en, eruption)) { |
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.
Nice!
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.
LGTM
Another step towards #6303's completion!
Hazardous Liquid Pools
For map makers: Place this in tiles with water. The level of the Hazardous Liquid Pool will control if it should kick up damaging spray in high winds (not implemented yet) and/or move with water flow (not implemented yet either). You probably shouldn't place this on its own - in order to easily implement the "act as water for movement" the Hazardous Liquid Pool relies on being placed in water.
TODO: