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 c:animal_foods tag #4080

Merged
merged 5 commits into from
Sep 10, 2024
Merged

Conversation

TelepathicGrunt
Copy link
Contributor

A tag to collect all the new mojang food tags into one parent tag so modders can use this to validate if an item is a feed item for some unknown animal. May make sorting easier or allow stocking up animal feeding machines without non-feedable items slipping in. Railcraft Reborn had a specific use case for this (see Neo PR)

Closes: #4036

Neoforge PR: neoforged/NeoForge#1465

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.

Change is good, the build is failing as one of the resource files isnt matching, please try re-running the datagen.

@TelepathicGrunt
Copy link
Contributor Author

TelepathicGrunt commented Sep 10, 2024

@modmuss50 I ran the datagen for the convention tags v2 module and no change. I don't see what specific resource is not matching
image
image

@modmuss50
Copy link
Member

Ill take a look, I should improve the github actions to actually show the diff...

@modmuss50 modmuss50 added the merge me please Pull requests that are ready to merge label Sep 10, 2024
@modmuss50
Copy link
Member

It still failed!

@modmuss50 modmuss50 merged commit 3d2379b into FabricMC:1.21.1 Sep 10, 2024
4 checks passed
modmuss50 pushed a commit that referenced this pull request Sep 10, 2024
* Add `c:animal_foods` tag

* checkstyle

* Spotless

* Add to lang generator

* Actually use the generated lang file

---------

Co-authored-by: modmuss50 <modmuss50@gmail.com>
(cherry picked from commit 3d2379b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge me please Pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

A tag to group up all animal food tags
2 participants