Skip to content

Conversation

@serprex
Copy link
Contributor

@serprex serprex commented Apr 17, 2025

Copy link
Contributor

@leggettc18 leggettc18 left a comment

Choose a reason for hiding this comment

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

Good work! I've got a couple of things I'd like you to change but in general I like what I'm seeing here!

@serprex serprex force-pushed the shipinit-timesavers branch from 03df490 to 8c00b4e Compare May 28, 2025 00:19
@serprex
Copy link
Contributor Author

serprex commented Jun 3, 2025

This PR is a little hairy, but it moves in the right direction. timesaver_hook_handlers.cpp is the real beast I'd like to start chipping away at. I'd rather merge this as-is then let it sit waiting for improvements which aren't clear to me how to implement

@serprex serprex force-pushed the shipinit-timesavers branch from e1c8ab1 to 0f5c77e Compare June 3, 2025 04:07
Copy link
Contributor

@Pepper0ni Pepper0ni left a comment

Choose a reason for hiding this comment

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

Did some testing, and everything seems to work, except for pre-existing essues that are out of scope for a simply hook change

@Malkierian Malkierian dismissed leggettc18’s stale review June 18, 2025 20:26

Changes addressed.

@Malkierian Malkierian merged commit 81b3123 into HarbourMasters:develop Jun 19, 2025
6 checks passed
krazyjakee pushed a commit to krazyjakee/OOT that referenced this pull request Sep 6, 2025
* timesavers: shipinit

* feedback

* intro skip not unconditional
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.

4 participants