Skip to content

Comment inactive warp in SilphCo1F - #591

Merged
Rangi42 merged 4 commits into
pret:masterfrom
PokefanMarcel:patch-2
Aug 10, 2026
Merged

Rangi42 merged 4 commits into
pret:masterfrom
PokefanMarcel:patch-2

Conversation

@PokefanMarcel

Copy link
Copy Markdown
Contributor

No description provided.

@CreamElDudJafar CreamElDudJafar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think there a few warps inactive or unused in silph. I think 11F has one and maybe 3rd or 7th floor too. Maybe comment them all inactive?

@Rangi42 Rangi42 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yeah, if this isn't a one-off, please use this PR to comment the rest of the inactive events.

@Rangi42
Rangi42 marked this pull request as draft June 28, 2026 17:05
@PokefanMarcel

Copy link
Copy Markdown
Contributor Author

I commented the one on SilphCo11F. Nothing on SilphCo3F or SilphCo7F as far as I can see. Also commented one in CeladonCity, that's the only one I could think of on the top of my head. If you know more I'm happy to add them.

@Rangi42 Rangi42 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Changed to "inaccessible" to match the comment in Gen 2's OlivineCity.asm.

Comment thread data/maps/objects/SilphCo1F.asm Resolved
Comment thread data/maps/objects/SilphCo11F.asm Resolved
Comment thread data/maps/objects/CeladonCity.asm Resolved
Co-authored-by: Rangi <35663410+Rangi42@users.noreply.github.com>
@Rangi42
Rangi42 marked this pull request as ready for review August 10, 2026 15:52

@Rangi42 Rangi42 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@Rangi42
Rangi42 merged commit 87f9f48 into pret:master Aug 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants