Skip to content
This repository has been archived by the owner on May 13, 2021. It is now read-only.

Remove redundant "seeds" tag #36

Open
wants to merge 2 commits into
base: 1.16_fabric
Choose a base branch
from

Conversation

hugeblank
Copy link

My justification for this PR is two-fold.

  1. Use of tags should be minimized as it pushes a standard that, in this case, really does not need to be applied. We can easily figure out which item is the seeds item by looking for an item that is a BlockItem, and comparing the resulting block from that item with the crop block.
  2. An entire config file is already provided to the end user, which needs changing anyways if implementing modded blocks. Having two places in which the end user needs to do changing is redundant, especially considering point one.

If you're an end user and desperately want this change before it gets merged (or closed) I've published a release on my fork here

hugeblank added 2 commits May 12, 2021 14:09
- Fixes TehNut-Mods#35

seed can be found by looking at BlockItem - removing the need for the tag
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant