Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added explicit roof to garage 1 #73983

Merged
merged 1 commit into from
May 21, 2024
Merged

Conversation

PatrikLundell
Copy link
Contributor

@PatrikLundell PatrikLundell commented May 21, 2024

Summary

None

Purpose of change

Add explicit roof to garage expansion version 1 rather than relying on magic.

Describe the solution

Add roof chunks to complement the floor ones and use them.
Also updated the flags to get rid of flowers and boulders inside and error messages about those.

Describe alternatives you've considered

Keep the expansion look exactly as before, with varying kinds of roofs depending on the roof elements of the constructed components. Went with uniform shingle roof instead.

Testing

  • Loaded a convenient save.
  • Surveyed the expansion.
  • Set the companion to build everything.
  • Looked at the result to verify it looks as expected.
    Screenshot (368)
    Screenshot (369)

The boulder in the wall isn't real, but rather the fairly recently introduced selective map amnesia.

Additional context

Did not experience sudden companion based glutton death this time, despite having to run the tests twice (screwed up by failing to convert floor to roof for a couple of tiles when generating the roof layouts). Assume that weird thing was a fluke.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels May 21, 2024
@Maleclypse Maleclypse merged commit 5ad0c83 into CleverRaven:master May 21, 2024
27 checks passed
@PatrikLundell PatrikLundell deleted the garage1 branch May 22, 2024 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants