Skip to content

Don't destroy immature netherwart with crop filter. Fixes #298 #299

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

Closed
wants to merge 0 commits into from

Conversation

jshipley
Copy link
Contributor

The netherwart block isn't actually a crop block (probably because bone meal doesn't work on it).

This PR changes the immature crop test for the block destroyer and enderic laser when a crop filter addon is present so that immature netherwart will not be destroyed.

@Rearth Rearth self-assigned this Mar 25, 2025
@jshipley
Copy link
Contributor Author

Cocoa blocks should also be accounted for. I can't think of any other minecraft croplike blocks that aren't subclasses of CropBlock.

@jshipley
Copy link
Contributor Author

jshipley commented Apr 18, 2025

It looks like this was closed accidentally when I reset my 1.21 branch (I forgot to "git checkout" after "git branch" when I made the new branch for this PR). I don't see any option to change the branch that this compares against.

Are you able to change this to pull the jshipley/netherwart_crop_filter branch, or should I just open a new PR?

@jshipley
Copy link
Contributor Author

I'm just going to open a new one from the correct branch...

@jshipley jshipley mentioned this pull request Apr 18, 2025
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

Successfully merging this pull request may close these issues.

2 participants