-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Removes Mortar Recipe's cement recipe book requirement #75348
Removes Mortar Recipe's cement recipe book requirement #75348
Conversation
This would be an ideal way to resolve this. If you want me to I can update your PR with this and fix the itemgroups that would need updating as well. |
Yes please! I agree that this is ideal. |
I really appreciate you putting a PR together for this. Thank you for doing this and I hope to see you contribute some more :) |
shortens 3 lines to one line Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Summary
Balance "Mortar is now an autolearned recipe"
Purpose of change
Fixes #74015.
Describe the solution
Makes Mortar into an autolearned recipe, and removes book requirement.
Describe alternatives you've considered
As many commenters in #74015 agree, simply making this into an autolearned recipe is a little pathetic. I'd love to revisit this once I have more experience
Testing
Linting produced no errors I can tell. I can't test through Docker currently; testing welcome!!!
Additional context
This is my first PR! Feedback appreciated