-
Notifications
You must be signed in to change notification settings - Fork 657
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
[Yoshi's Island] Minor Fixes #3142
Conversation
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.
Changes LGTM. Merged into main
and it generates. Only other instance of "guranteed" is in CV64 which is outside the scope of this
Personal comment, 10-Ups were always supposed to be classed as useful, it seems when I was refactoring the classification assignment, they accidentally got marked as filler and slipped past my notice. |
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.
Delete the apyi file you accidentally attached to this PR
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.
Looks fine, other than that extra file that shouldn't be in the PR
Please format your title with what portion of the project this pull request is
targeting and what it's changing.
ex. "MyGame4: implement new game" or "Docs: add new guide for customizing MyGame3"
What is this fixing or adding?
-Changed '10-Up' classification to Useful
-Fixed option typo from 'bosses_guranteed' to 'bosses_guaranteed'
How was this tested?
I generated after changing them to make sure they worked.
If this makes graphical changes, please attach screenshots.