-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Fixed Forecast and Flower Gift #2978
Fixed Forecast and Flower Gift #2978
Conversation
Ready for review! I added a missing interaction with Air Lock (that also makes it pretty much impossible to test Strong Winds), reworked how the weather form reversion is done, and added all the missing tests. Castform is now functional with snow, too! |
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.
B_WEATHER_FORMS
already exists.
In retrospect, I didn't properly communicate that I wanted Castform and Cherrim's generational changes to be handled by the form change tables, which is why I proposed the optional ability parameter, in order to toggle between them by changing the data.
Added assumptions, ready again. Ignore the force push, I forgot committed code review changes don't just magically make their way onto my local repo, but the change was made in the new commit anyways so I just force pushed. |
Description
This PR does the following:
u8 TryWeatherFormChange
has been removed (last remnant of old Castform hacks?).Tests Added
Issue(s) that this PR fixes
Fixes #2972.
Discord contact info
Agustin#1522