-
Notifications
You must be signed in to change notification settings - Fork 423
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] Sync remaining c
tags with NeoForge
#4161
Open
TelepathicGrunt
wants to merge
37
commits into
FabricMC:1.21.1
Choose a base branch
from
TelepathicGrunt:SyncWithNeoforgeTag
base: 1.21.1
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
3cb6741
Add more chest tags
TelepathicGrunt d85d74f
adjust temperate biome tag
TelepathicGrunt 9485c05
More Cobblestone tags
TelepathicGrunt 87cd331
end stones tag
TelepathicGrunt 6f58e17
Gravels tag
TelepathicGrunt 832dc06
Netherracks tag
TelepathicGrunt 07b096d
Add all ores tags for consistency
TelepathicGrunt a07ebd6
Ore rates, ore in ground, and ore bearing tags added
TelepathicGrunt cff626f
Added sands tags
TelepathicGrunt 2f6001a
added bones tag
TelepathicGrunt 62e1be5
Added eggs tag
TelepathicGrunt 1508cc6
Added feathers tag
TelepathicGrunt 117c8ca
Added gunpowders tag
TelepathicGrunt 710fec6
Added mushrooms tag
TelepathicGrunt c4af34a
Added nether stars tag
TelepathicGrunt 43c6198
Added seeds tags
TelepathicGrunt d5f8cfb
Added fence and fence gate tags
TelepathicGrunt 88e8adc
Added nether and end empty category tags
TelepathicGrunt b708729
Added lush biome tag
TelepathicGrunt 1994820
Added magical tag
TelepathicGrunt 025acf8
Added rare biome tag
TelepathicGrunt 2fcd4d7
Added plateau tag
TelepathicGrunt 38efc6b
added spooky biome tag
TelepathicGrunt 7653b61
Added sandy tag
TelepathicGrunt 60a6414
Fixed checkstyle issues
TelepathicGrunt 611b66e
Run datagen
TelepathicGrunt 5236d2d
more javadoc and fixed biomes in temperature tag
TelepathicGrunt 63f469c
fixed typo in biome temperature tag javadoc
TelepathicGrunt 5d0a7ae
attempt at potion tag javadoc
TelepathicGrunt b98b9b3
Add bottled potion subtag
TelepathicGrunt ef27c6c
Datagen bottled potion and biome temperature fix
TelepathicGrunt f006a2b
Improved potion javadoc more
TelepathicGrunt 7af6e54
Changed potion verb to noun
TelepathicGrunt ffb18fa
Improve potion javadoc to mention data component
TelepathicGrunt a4941f9
Merge branch '1.21.1' into SyncWithNeoforgeTag
TelepathicGrunt 3477bd7
special exception for certain biome tag
TelepathicGrunt 207fd19
checkstyle
TelepathicGrunt File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Im not sure about the rare biomes or the ore rates, as they have nothing to do with the tags. You could have a another datapack that invalidates these. Whats a use case?
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.
Much like storage block tags that can be invalidated by datapacks, it is on the modpack maker to adjust any relevant tags when they are customizing their modpacks so personally, that angle, I think is fine.
For biome rarity, some mods seems to target these biomes for uncommon ores or mobs. Not many but a few. Like Alex's Mobs seems to add Mungus to these rare tagged biomes. There's Create Arcane Engineering that adds Aurum, Ferrum, and other ores to rare biomes. Resourceful Bees spawns their easter egg Oreo Bee in rare biomes. Ice and Fire spawns their Pixie Village in rare biomes as well. So there's some use cases for targeting biome that are infrequent/difficult to find by default. And then if pack makers make a biome more or less common, they would adjust the is_rare tag to reflect the new change,
For ore_rates, I had the same thought as you of who even uses this. Turns out Tinker's uses it quite a bit. I'll ping @KnightMiner to come talk about his usages for that. RCXcrafter said he should be using ore rates tag but haven't done so yet. If packs changes the rates, they should adjust the tags too.