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

ALTTP: remove excess Blue Mail items from (non-progressive) hard item pool #2046

Merged
merged 1 commit into from
Jul 28, 2023

Conversation

el-u
Copy link
Collaborator

@el-u el-u commented Jul 28, 2023

What is this fixing or adding?

https://discord.com/channels/731205301247803413/1133934296122544200/1133934296122544200
With the specific setting combination of non-progressive armors plus hard item pool, a total of 4 "Blue Mail" items are placed in the multiworld.
Appropriate for this mode would be to have just 2 armor items in the pool.

How was this tested?

Generating a game with the minimal reproducible yaml

name: YourName{number}
game: A Link to the Past
A Link to the Past:
  progressive: off
  item_pool: hard

Before: 4x Blue Mail in the generated game
After: 2x Blue Mail in the generated game (+2 additional rupee items were generated)

@el-u el-u requested a review from Berserker66 as a code owner July 28, 2023 10:33
@Berserker66 Berserker66 merged commit 0d28eeb into ArchipelagoMW:main Jul 28, 2023
12 checks passed
@el-u el-u deleted the too_blue branch July 28, 2023 12:21
@ThePhar ThePhar changed the title alttp: remove excess Blue Mail items from (non-progressive) hard item pool ALTTP: remove excess Blue Mail items from (non-progressive) hard item pool Jul 28, 2023
@ThePhar ThePhar added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants