style: Format both flavors of RetroDays #4486
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The line count on the original Retrodays reformatting will be immense, I can already predict it
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose of change
I noticed, while looking through the tileset config files for a completely unrelated reason, that the original RetroDays tileset config file was HORRIBLY formatted for some reason. For the sake of improving the keeping of style within the code at least a little bit, I have let the auto-formatter have its way with it to change it from its one-line eldritch horror nature to a respectably formatted JSON file.
Also while I was at it ran the autoformatter on the other tileset config files, and RetroDays+ had some minor formatting errors that I also found.
Describe the solution
Formats the tileset config JSON file of RetroDays and RetroDays+ to our standards
Describe alternatives you've considered
Reducing the amount of spaghetti 'code' is always good, and it's not like this could possibly harm anything
Testing
Confirmed that the game can indeed still read RetroDays even when it's properly formatted.
Additional context
The two people who use RetroDays, rejoice!