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 error if spawn placements are missing for entities that spawn in-world #689

Open
wants to merge 3 commits into
base: 1.20.x
Choose a base branch
from

Conversation

ThatGravyBoat
Copy link

Description

This PR adds an error in dev if there are any entities added to the MobSpawnSettings for biomes and the entity does not register a spawn placement. The reason for this is that if you do not register a spawn placement the entity can spawn anywhere which includes in the are and is usually a bug 99.9% of the time and can be hard to track down if a developer does not know where to look and does not know they need to register to such an event this logging will help with such an issue and help prevent such bug from reaching production. If a developer does need to not have their entities restricted they can register with NO_RESTRICTIONS as what vanilla does if none is registered is just default to NO_RESTRICTIONS and always true for the predicate.

@CLAassistant
Copy link

CLAassistant commented Mar 5, 2024

CLA assistant check
All committers have signed the CLA.

@neoforged-pr-publishing
Copy link

  • Publish PR to GitHub Packages

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants