Skip to content

[CHORE] Correct event cancelable status comments in ScriptEventType#7602

Open
ahmed7p wants to merge 1 commit into
FunkinCrew:developfrom
ahmed7p:EV-Fix
Open

[CHORE] Correct event cancelable status comments in ScriptEventType#7602
ahmed7p wants to merge 1 commit into
FunkinCrew:developfrom
ahmed7p:EV-Fix

Conversation

@ahmed7p
Copy link
Copy Markdown

@ahmed7p ahmed7p commented May 20, 2026

Description

This PR updates the documentation comments inside ScriptEventType.hx to accurately reflect the actual engine behavior regarding script events.

Key Changes

  • Clarified Cancelable Status: Updated events like GAME_OVER, SONG_END, NOTE_HOLD_DROP, SONG_BEAT_HIT, SONG_STEP_HIT, and _BEGIN state events to explicitly state they IS cancelable, aligning them with their implementation and logic checks in the source code.
  • Fixed Musical Typo: Corrected SONG_BEAT_HIT description from "once per step" to "once per beat".

This ensures modders and custom script creators have an accurate API reference when working with the engine's event system.

@github-actions github-actions Bot added status: pending triage Awaiting review. pr: haxe PR modifies game code. size: medium A medium pull request with 100 or fewer changes. and removed status: pending triage Awaiting review. labels May 20, 2026
@NotHyper-474 NotHyper-474 added status: pending triage Awaiting review. type: chore Involves minor adjustments that do not make functional changes. labels May 20, 2026
@Hundrec Hundrec changed the title [CHORE] Correct event cancelable status in ScriptEventType [CHORE] Correct event cancelable status comments in ScriptEventType May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: haxe PR modifies game code. size: medium A medium pull request with 100 or fewer changes. status: pending triage Awaiting review. type: chore Involves minor adjustments that do not make functional changes.

Development

Successfully merging this pull request may close these issues.

2 participants