Skip to content

Fixing Debug/Map Renderer actions#322

Closed
GreyGrisGrey wants to merge 9 commits into
michaelchessall:masterfrom
GreyGrisGrey:testfixB
Closed

Fixing Debug/Map Renderer actions#322
GreyGrisGrey wants to merge 9 commits into
michaelchessall:masterfrom
GreyGrisGrey:testfixB

Conversation

@GreyGrisGrey
Copy link
Copy Markdown
Contributor

About the PR

Added a couple small checks to a couple functions to allow github actions to run that were previously consistently returning errors. Relatedly, updated some guidebook/status effect names to reduce warnings and more closely follow naming conventions.

Technical details

Added null check to SpawnPlayer function preventing some null reference errors.
Added contains check to a test also preventing a number of errors.
Added lines to allow certain github actions to be run manually, making testing easier.
Added ftl file containing localizations for some status effects that didn't have them, reducing warnings by a substantial amount.
Updated numerous factions/status names to work with ftl file formatting and more closely match naming conventions.

Requirements

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.

1 participant