-
-
Notifications
You must be signed in to change notification settings - Fork 194
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 a biome villager types data map to replace VillagerType#BY_BIOME
#1879
Add a biome villager types data map to replace VillagerType#BY_BIOME
#1879
Conversation
This PR adds a `neoforge:biome_villagers` biome data map to replace `VillagerType#BY_BIOME`, allowing mods to change the type of the villagers spawned in their own biomes. To avoid a sudden breaking change, the map will still be used for now, but the fallback to it will be removed in 1.22.
Last commit published: aa5a0889d9e2b118cde82f38d6eb3e81fd2414b1. PR PublishingThe artifacts published by this PR:
Repository DeclarationIn order to use the artifacts published by the PR, add the following repository to your buildscript: repositories {
maven {
name 'Maven for PR #1879' // https://github.com/neoforged/NeoForge/pull/1879
url 'https://prmaven.neoforged.net/NeoForge/pr1879'
content {
includeModule('net.neoforged', 'neoforge')
includeModule('net.neoforged', 'testframework')
}
}
} MDK installationIn order to setup a MDK using the latest PR version, run the following commands in a terminal. mkdir NeoForge-pr1879
cd NeoForge-pr1879
curl -L https://prmaven.neoforged.net/NeoForge/pr1879/net/neoforged/neoforge/21.4.79-beta-pr-1879-biome-villagers-data-map/mdk-pr1879.zip -o mdk.zip
jar xf mdk.zip
rm mdk.zip || del mdk.zip To test a production environment, you can download the installer from here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good I would just add somewhere stating the default value is plains villager type so people can easily see that
VillagerType#BY_BIOME
VillagerType#BY_BIOME
Any movement on this? |
@neoforged/bots backport 1.21.1 |
Backporting to |
@Matyrobbrt backport to 1.21.1 failed. Click for failure reasonCommand 'git apply --ignore-whitespace --recount -C0 __diff' failed execution with error: error: patch failed: src/main/java/net/neoforged/neoforge/common/data/internal/NeoForgeDataMapsProvider.java:24 https://github.com/neoforged/PRActionRunner/actions/runs/13101887154 |
🚀 This PR has been released as NeoForge version |
This PR adds a
neoforge:villager_types
biome data map to replaceVillagerType#BY_BIOME
, allowing mods to change the type of the villagers spawned in their own biomes.To avoid a sudden breaking change, the map will still be used for now, but the fallback to it will be removed in 1.22.
See #1875 for context of making this PR.