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

Several tag fixes and additions #3912

Merged
merged 7 commits into from
Jul 7, 2024

Conversation

TelepathicGrunt
Copy link
Contributor

@TelepathicGrunt TelepathicGrunt commented Jul 5, 2024

Changes includes:

  • Renaming all food tags under c:foods/ to be singular. Example is c:foods/fruits to be c:foods/fruit. This is because you would say Fruit Foods and not Fruits Foods. Basically, foods is the noun. Fruit is the descriptor of the food noun. Backwards compat added here to reduce breakage for people.

  • Added c:foods/golden to collect all gold based foods. Which then can be paired ingredient intersections and differences to target or exclude from the other food tags.

  • Added c:tools/igniter to now have Flint and Steel be considered a tool properly.

  • Adding c:ender_pearls, c:glass_blocks/colorless, c:glass_blocks/cheap, c:glass_blocks/tinted, c:glass_panes/colorless tags to match what Neoforge has closer.

Neoforge PR: neoforged/NeoForge#1238

Closes: #3899

Closes: #3869

Closes half of : #3895

Copy link
Member

@modmuss50 modmuss50 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for the contiuned support with this.

@modmuss50 modmuss50 added enhancement New feature or request last call If you care, make yourself heard right away! priority:medium Medium priority PRs that should get reviews merge me please Pull requests that are ready to merge labels Jul 6, 2024
@TelepathicGrunt
Copy link
Contributor Author

@modmuss50 One second, I'll deal with the merge conflicts

@TelepathicGrunt
Copy link
Contributor Author

@modmuss50 Should be good now

@modmuss50
Copy link
Member

Lol, I was doing the same :D Wondered why I couldnt push it lol. Thanks 👍

@modmuss50 modmuss50 merged commit fb44a03 into FabricMC:1.21 Jul 7, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request last call If you care, make yourself heard right away! merge me please Pull requests that are ready to merge priority:medium Medium priority PRs that should get reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fabric Standard Tags for Food are limited. Missing a few c tags,
2 participants