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

Lingo: Sync config with game update #2447

Merged
merged 4 commits into from
Nov 16, 2023
Merged

Conversation

hatkirby
Copy link
Collaborator

@hatkirby hatkirby commented Nov 10, 2023

What is this fixing or adding?

There is an update to the game that slightly changes the topography of the world, so this I've changed the world config to be up to date with that.

Also fixed a warning in the validate config script that should've been handled in an earlier PR.

I asked the dev if he had any other plans to make topological changes soon and he said no.

How was this tested?

pytest and a handful of test runs on different settings.

If this makes graphical changes, please attach screenshots.

Also fixed a warning in the validate config script that should've been handled in an earlier PR.
Fixing this actually has no real effect, but it's good to keep the map accurate.
This actually does matter for logic.
@ThePhar ThePhar added the is: maintenance Regular updates to requirements and utilities that do not fix bugs or change/add features. label Nov 10, 2023
@hatkirby hatkirby marked this pull request as ready for review November 11, 2023 00:11
@black-sliver
Copy link
Member

Do I understand correctly that without this change, rolling on main creates a seed that has different logic than the actual game?

@hatkirby
Copy link
Collaborator Author

Correct.

@black-sliver black-sliver added the affects: release/blocker Issues/PRs that must be addressed before next official release. label Nov 15, 2023
@ThePhar ThePhar merged commit a4b625c into ArchipelagoMW:main Nov 16, 2023
7 checks passed
@hatkirby hatkirby deleted the game-update branch November 19, 2023 14:58
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects: release/blocker Issues/PRs that must be addressed before next official release. is: maintenance Regular updates to requirements and utilities that do not fix bugs or change/add features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants