Skip to content

Add glitch practice cheats#2570

Merged
garrettjoecox merged 8 commits intoHarbourMasters:developfrom
Pepe20129:glitch_practice_cheats_3
Aug 20, 2023
Merged

Add glitch practice cheats#2570
garrettjoecox merged 8 commits intoHarbourMasters:developfrom
Pepe20129:glitch_practice_cheats_3

Conversation

@Pepe20129
Copy link
Contributor

@Pepe20129 Pepe20129 commented Mar 2, 2023

Adds a cheat to prevent fish from despawning, a cheat to prevent bugs from despawning and a cheat to change the fuse timer of bombs.
Meant to help with practicing glitches.

Build Artifacts

@Pepe20129 Pepe20129 changed the title Add fish don't despawn cheat Add glitch practice cheats Mar 3, 2023
@briaguya0
Copy link
Contributor

this isn't a bugfix so it should be pointed to develop, not khan

@Pepe20129 Pepe20129 changed the base branch from develop-khan to develop March 7, 2023 16:10
@briaguya0 briaguya0 added the merge conflicts PR has conflicts that need to be resolved before it can be merged label Apr 3, 2023
Copy link
Contributor

@garrettjoecox garrettjoecox left a comment

Choose a reason for hiding this comment

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

LGTM, just needs a rebase

Edit: Also can you add the new cvars to the presets.h under enhancmentCvars

@Pepe20129 Pepe20129 force-pushed the glitch_practice_cheats_3 branch from 5265887 to c25f682 Compare August 8, 2023 14:48
@Pepe20129
Copy link
Contributor Author

I think I rebased correctly but it's my first time rebasing a pr so I'm not sure.
I added the cvars to cheatCvars instead of enhancementsCvars.

@garrettjoecox garrettjoecox added merge ready and removed merge conflicts PR has conflicts that need to be resolved before it can be merged merge ready labels Aug 11, 2023
@garrettjoecox garrettjoecox merged commit d06e6af into HarbourMasters:develop Aug 20, 2023
@Pepe20129 Pepe20129 deleted the glitch_practice_cheats_3 branch August 20, 2023 22:01
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