Skip to content

Updates to fix YAML linter's complaining#320

Merged
michaelchessall merged 1 commit into
michaelchessall:masterfrom
GreyGrisGrey:linterpatch
May 1, 2026
Merged

Updates to fix YAML linter's complaining#320
michaelchessall merged 1 commit into
michaelchessall:masterfrom
GreyGrisGrey:linterpatch

Conversation

@GreyGrisGrey
Copy link
Copy Markdown
Contributor

About the PR

Modified/added a number of yml/ftl files to reduce YAML linter errors. Linter has no errors on my machine

Technical details

Added missing bounty name localizations
Added localizations for faction guidebook entries and changed faction guidebook ids to work with the ftl file
Some bounty descriptions were disconnected from existing bounties, they should now be linked
Bounties with no descriptions had placeholders added. I don't think bounty descriptions are currently visible so I don't believe this should cause problems
Updated two image sources as the old sources appear to have been moved or deleted at some point
Removed markingCategory attribute from persistence additions to human_hair.yml. The markingCategory attribute is used in exactly one largely disconnected go file that was last updated 4 years ago and nowhere else, and furthermore nothing else in the human_hair.yml file has it. I don't believe this removal will cause issues
Re added antagonist guidebook entries for the sole purpose of removing linter errors

Requirements

Added missing bounty name localizations
Added missing faction localizations
Attached bounty descriptions to proper bounties, added placeholders where no description existed
Changed guidebook ids to work with localizations
Added localizations for faction guidebook entries
Updated two images
@michaelchessall michaelchessall merged commit c25e8e3 into michaelchessall:master May 1, 2026
13 of 15 checks passed
@GreyGrisGrey GreyGrisGrey deleted the linterpatch branch May 1, 2026 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants