Skip to content

Pokemon Emerald: Added Pokemon Gen 3 Adjuster data#5145

Merged
NewSoupVi merged 4 commits intoArchipelagoMW:mainfrom
RhenaudTheLukark:emerald-main
Sep 1, 2025
Merged

Pokemon Emerald: Added Pokemon Gen 3 Adjuster data#5145
NewSoupVi merged 4 commits intoArchipelagoMW:mainfrom
RhenaudTheLukark:emerald-main

Conversation

@RhenaudTheLukark
Copy link
Contributor

What is this fixing or adding?

This pull request adds the data related to the Pokémon Gen 3 Adjuster made by myself for Pokémon Emerald.
@Zunawe, the world maintainer of Pokémon Emerald, has asked me to move a PR that was originally made to his own repository to the main Archipelago repository.
The code added here will not have any impact on its own, as it needs PR #5085 to be merged to be used.

How was this tested?

Manually, over many asyncs.

@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Jun 28, 2025
@Exempt-Medic Exempt-Medic added the is: enhancement Issues requesting new features or pull requests implementing new features. label Jun 28, 2025
@Exempt-Medic Exempt-Medic requested a review from Zunawe June 28, 2025 22:48
Copy link
Collaborator

@Zunawe Zunawe left a comment

Choose a reason for hiding this comment

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

Two easy comments, and once those are addressed, LGTM.

I didn't scrutinize the constants file, but I doubt I need to care for it to work correctly and not break anything. Just data. I skimmed the docs, and it looks fine in terms of English, formatting, correctness, etc. afaict.

If I have any substantial comments about the function or format of the constants file, I'll just address them in the actual adjuster PR when I get to it.

Copy link
Collaborator

@Zunawe Zunawe left a comment

Choose a reason for hiding this comment

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

The extracted data changes also came directly from me, so I trust it. All it does is add new rom addresses. Nothing is modified.

@Zunawe Zunawe 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 Jul 6, 2025
@NewSoupVi NewSoupVi merged commit a0a1c5d into ArchipelagoMW:main Sep 1, 2025
3 checks passed
Crazycolbster pushed a commit to Crazycolbster/Archipelago that referenced this pull request Nov 18, 2025
* Added Pokemon Gen 3 Adjuster data

* Updated extracted data

* Commented out adjuster docs for now

* Replace <b> in the docs markers with **
eternalcode0 pushed a commit to eternalcode0/Archipelago that referenced this pull request Dec 20, 2025
* Added Pokemon Gen 3 Adjuster data

* Updated extracted data

* Commented out adjuster docs for now

* Replace <b> in the docs markers with **
eternalcode0 pushed a commit to eternalcode0/Archipelago that referenced this pull request Dec 21, 2025
* Added Pokemon Gen 3 Adjuster data

* Updated extracted data

* Commented out adjuster docs for now

* Replace <b> in the docs markers with **
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