Skip to content

[BUGFIX] Prevent crash when missing credits.json with -D NO_HARDCODED_CREDITS#5982

Merged
Hundrec merged 1 commit into
FunkinCrew:dummy/develop-v0.7.5from
macohi:sphis/fix-missing-credits-json
Oct 14, 2025
Merged

[BUGFIX] Prevent crash when missing credits.json with -D NO_HARDCODED_CREDITS#5982
Hundrec merged 1 commit into
FunkinCrew:dummy/develop-v0.7.5from
macohi:sphis/fix-missing-credits-json

Conversation

@macohi
Copy link
Copy Markdown
Contributor

@macohi macohi commented Sep 11, 2025

Linked Issues

N/A

Description

When the game is compiled with the flag -D NO_HARDCODED_CREDITS the game will crash when trying to enter the credits state because of the missing credits json.

This is cause the credits json is in the excludes folder, I mean it would be hardcoded credits so I guess it makes sense? But there isn't any code or checks for un-hardcoded credits.

This PR fixes that.

Screenshots/Videos

Before:

Screen.Recording.2025-09-11.174910.mp4

After:

Screen.Recording.2025-09-11.181030.mp4

@github-actions github-actions Bot added status: pending triage Awaiting review. size: small A small pull request with 10 or fewer changes. labels Sep 11, 2025
@AbnormalPoof AbnormalPoof added type: major bug Involves a major bug, including crashes, softlocks, or issues blocking progression pr: haxe PR modifies game code. labels Sep 12, 2025
@Hundrec Hundrec changed the title [BUGFIX] Missing Credits.json with -D NO_HARDCODED_CREDITS [BUGFIX] Prevent crash when missing credits.json with -D NO_HARDCODED_CREDITS Sep 13, 2025
@github-actions github-actions Bot removed the pr: haxe PR modifies game code. label Sep 26, 2025
@AbnormalPoof AbnormalPoof added status: reviewing internally Under consideration and testing. and removed status: pending triage Awaiting review. status: reviewing internally Under consideration and testing. labels Oct 14, 2025
@AbnormalPoof AbnormalPoof force-pushed the sphis/fix-missing-credits-json branch from 6c66fb7 to 7faa626 Compare October 14, 2025 01:31
@AbnormalPoof AbnormalPoof added the status: reviewing internally Under consideration and testing. label Oct 14, 2025
@Hundrec Hundrec added status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. and removed status: reviewing internally Under consideration and testing. labels Oct 14, 2025
@Hundrec Hundrec added this to the 0.7.6 milestone Oct 14, 2025
@Hundrec Hundrec changed the base branch from develop to dummy/develop-v0.7.5 October 14, 2025 02:15
@Hundrec Hundrec merged commit 6c098d5 into FunkinCrew:dummy/develop-v0.7.5 Oct 14, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: small A small pull request with 10 or fewer changes. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. type: major bug Involves a major bug, including crashes, softlocks, or issues blocking progression

Development

Successfully merging this pull request may close these issues.

3 participants