Skip to content
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

Pokemon Emerald: Clean up free fly blacklist #4552

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

Zunawe
Copy link
Collaborator

@Zunawe Zunawe commented Jan 25, 2025

What is this fixing or adding?

Cleans up what was added in #4165 and adds sphere 1 fly locations as default-blacklisted options.

How was this tested?

Generated and checked the free fly location in the debugger.

@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Jan 25, 2025
@Zunawe Zunawe added the is: enhancement Issues requesting new features or pull requests implementing new features. label Jan 25, 2025
Copy link
Contributor

@Jouramie Jouramie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ran a generation with the default values, seems to work fine. LGTM

@Zunawe
Copy link
Collaborator Author

Zunawe commented Jan 26, 2025

Default values on the blacklist, or on the yaml as a whole? The blacklist code doesn't run if free fly location isn't enabled, which I forgot when I first ran it to test.

@Jouramie
Copy link
Contributor

Jouramie commented Jan 26, 2025

With the default blacklist, I mean. I did enable the free fly location so it did run.

Copy link
Collaborator

@ScipioWright ScipioWright left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't comment on the validity of town names, but otherwise it looks like a pretty simple change.

@ScipioWright ScipioWright added waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer. and removed waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Jan 27, 2025
@Exempt-Medic Exempt-Medic merged commit b570aa2 into ArchipelagoMW:main Jan 27, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: enhancement Issues requesting new features or pull requests implementing new features. waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants