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

Add weeds to standard_domestic_landscaping_palette #79759

Merged

Conversation

Milopetilo
Copy link
Contributor

Summary

None

Purpose of change

I discovered there's weeds in the game, I'd assume most places would at least have a few around.

Describe the solution

Adds weeds to standard_domestic_landscaping_palette so they spawn instead of flowers sometimes.

Describe alternatives you've considered

I wanted to add the weeds to standard_domestic_palette to have them spawn on any t_region_groundcover_urban but I ran into the problem that sometimes . is used on a second story as t_open_air and it'd be a lot of trouble to fix and make sure no-one else in the future runs into the issue. As a result I have added weeds to . in standard_domestic_landscaping_palette since that is usually only used on ground floors, but that also failed :/

Testing

Game loads, weeds spawn, noticed no errors thrown.

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Feb 20, 2025
@Maleclypse Maleclypse merged commit 11ae75f into CleverRaven:master Feb 21, 2025
29 of 30 checks passed
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