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

Noita: Update to use new Options API #2370

Merged
merged 17 commits into from
Jan 19, 2024

Conversation

ScipioWright
Copy link
Collaborator

What is this fixing or adding?

Reworking the options to make it work with the new options API.
Also reworked stuff in several spots to use world: NoitaWorld instead of multiworld: MultiWorld

How was this tested?

Generated a few games with different settings, connected it to the mod once

If this makes graphical changes, please attach screenshots.

N/A

@ScipioWright ScipioWright added the is: refactor/cleanup Improvements to code/output readability or organizization. label Oct 26, 2023
worlds/noita/Rules.py Outdated Show resolved Hide resolved
worlds/noita/__init__.py Outdated Show resolved Hide resolved
worlds/noita/Regions.py Outdated Show resolved Hide resolved
@heinermann
Copy link
Collaborator

You should be able to delete the add_location function in Regions.py.

worlds/noita/Events.py Outdated Show resolved Hide resolved
worlds/noita/Items.py Outdated Show resolved Hide resolved
worlds/noita/Regions.py Outdated Show resolved Hide resolved
worlds/noita/Regions.py Outdated Show resolved Hide resolved
worlds/noita/Regions.py Outdated Show resolved Hide resolved
worlds/noita/Regions.py Outdated Show resolved Hide resolved
worlds/noita/Rules.py Outdated Show resolved Hide resolved
worlds/noita/__init__.py Show resolved Hide resolved
@ScipioWright ScipioWright marked this pull request as draft November 22, 2023 14:36
@ScipioWright
Copy link
Collaborator Author

Converting to draft until I have the time to fix the merge conflicts and deal with the comments

@ScipioWright ScipioWright marked this pull request as ready for review December 17, 2023 18:35
@Berserker66 Berserker66 merged commit 5f9ce2b into ArchipelagoMW:main Jan 19, 2024
12 checks passed
@ScipioWright ScipioWright deleted the option-changes branch January 30, 2024 14:08
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
Reworking the options to make it work with the new options API.
Also reworked stuff in several spots to use world: NoitaWorld instead of multiworld: MultiWorld
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: refactor/cleanup Improvements to code/output readability or organizization.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants