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

[1.21.1] Biomemodifications: Check to make sure Feature isn't present before adding it. BackPort of #4323 #4338

Closed
wants to merge 1 commit into from

Conversation

JT122406
Copy link
Contributor

Fixes an issue where a feature order cycle error will happen because a feature may be added to a biome twice. This can occur if a biome appears in more than one tag and you want a feature in biomes from both tags, currently it gets added twice to the biome causing a feature order cycle error

@modmuss50 modmuss50 added bug Something isn't working merge me please Pull requests that are ready to merge small change labels Dec 30, 2024
@modmuss50
Copy link
Member

Hi, just seen this PR is identical to #4323, it will be easier for me to manage if I cherry pick it down. We only need a seperate PR when the change cannot be easily chery picked. Thanks.

@modmuss50 modmuss50 closed this Dec 30, 2024
@JT122406
Copy link
Contributor Author

Sounds good I wasn't sure if you just cherry-picked or not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working merge me please Pull requests that are ready to merge small change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants