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

Should c:coal item tag be removed in favor of minecraft's coal tag? #3742

Open
TelepathicGrunt opened this issue Apr 28, 2024 · 4 comments
Open

Comments

@TelepathicGrunt
Copy link
Contributor

It is a bit late now but wanted to document it here so it is not forgotten about. Nouns should be plural so this tag not being called coals was a typo. It isn't in neo side so no mismatch there. But it can't be fixed on Fabric's side without it being a breaking change in code unless a second coals tag is made, deprecate the singular one, and pull in the singular one into the plural one as an optional entry.

@TelepathicGrunt TelepathicGrunt changed the title c:coal should technically be c:coals c:coal item tag should technically be c:coals Apr 28, 2024
@apple502j
Copy link
Contributor

Hmm, why is this coals in NeoForge? Coal is uncountable.

@Shnupbups
Copy link
Contributor

Hmm, why is this coals in NeoForge? Coal is uncountable.

I believe we decided that we still use the 's' even for uncountable nouns, since the tags collect different types of them, so it is sort of grammatically correct. Like you can have lots of fruit, but also lots of different fruits. You can have lots of coal, but also lots of different coals.

@TelepathicGrunt
Copy link
Contributor Author

What shnup said. It’s why the tag convention is fishes and meats because it’s easier and less likely to cause confusion on whether to have an s or not for a noun by going with thinking of the tags as different types of stuff which would then required the s. Thus all nouns should have s generally. If confused, add s.

The coal tag isn’t on neoforge side yet though at all. Might add it over there to match if I can find some neoforge mods with different coal types.

@TelepathicGrunt
Copy link
Contributor Author

: thonk :
Coals is a Mc tag

https://github.com/misode/mcmeta/blob/data-json/data/minecraft/tags/items/coals.json

Now I’m curious, what was fabric’s motivation originally to have a coal tag under c instead of using mc’s?

@TelepathicGrunt TelepathicGrunt changed the title c:coal item tag should technically be c:coals Should c:coal item tag be removed in favor of minecraft's coal tag? May 27, 2024
TelepathicGrunt added a commit to TelepathicGrunt/fabric that referenced this issue May 27, 2024
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

No branches or pull requests

3 participants