Skip to content

Added cancellable ChangeGameModeEvent support - #10

Merged
ShaneBeee merged 10 commits into
SkriptDev:dev/patchfrom
libnexus:dev/patch
Feb 5, 2026
Merged

Added cancellable ChangeGameModeEvent support#10
ShaneBeee merged 10 commits into
SkriptDev:dev/patchfrom
libnexus:dev/patch

Conversation

@libnexus

@libnexus libnexus commented Feb 5, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

Added game mode support preliminarily including GameMode type and serialization/deserialization, OnGameModeChange event and context values (old and new game mode), added cancellation support, and included name of support for game mode objects.


Target Hytale Versions: any
Requirements: none
Related Issues: TODO

Checklist before requesting a review

…ialization/deserialization, OnGameModeChange event and context values (old and new game mode), added cancellation support, and included name of support for game mode objects.

@ShaneBeee ShaneBeee 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.

Sorry I dont have a style guide setup, so this is just a bunch of nitpicks.
Looks good otherwise.

Comment thread src/main/java/com/github/skriptdev/skript/plugin/elements/types/TypesServer.java Outdated
Comment thread src/main/java/com/github/skriptdev/skript/plugin/elements/types/TypesServer.java Outdated
libnexus and others added 8 commits February 5, 2026 10:29
…ts/player/EvtPlayerChangeGameMode.java

Co-authored-by: Shane Bee <shanebolenback@me.com>
…ts/player/EvtPlayerChangeGameMode.java

Co-authored-by: Shane Bee <shanebolenback@me.com>
…ts/player/EvtPlayerChangeGameMode.java

Co-authored-by: Shane Bee <shanebolenback@me.com>
…r. Removed serializer from type registration and changed to INSERT VERSION
@libnexus
libnexus requested a review from ShaneBeee February 5, 2026 12:24
@libnexus

libnexus commented Feb 5, 2026

Copy link
Copy Markdown
Contributor Author

I've made the changes, and you're right. IntelliJ is very insistent about "optimizing" imports. No worries about no style guide since it's knew, but thanks for letting me know regardless.

@ShaneBeee

Copy link
Copy Markdown
Member

IntelliJ is very insistent about "optimizing" imports.

Not a worry. I didnt have an editoconfig setup but I do now.

@ShaneBeee ShaneBeee 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.

Looks good to me.
Thank you for your contribution.

@ShaneBeee
ShaneBeee merged commit f00174b into SkriptDev:dev/patch Feb 5, 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.

2 participants