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

Helper functions for common crafting recipes #594

Merged
merged 9 commits into from
Apr 9, 2024

Conversation

kirderf1
Copy link
Member

@kirderf1 kirderf1 commented Apr 6, 2024

Introduces CommonRecipes with helper functions for creating common recipes during data-generation.

The new helper functions are also being used where appropriate.
Cruxite recipes originally had raw cruxite as their unlock criteria, but not the actual block ingredient. For the recipes involved in this PR, I also added the block ingredient as an alternative unlock criteria.

I was originally thinking of going over additional recipes, like doors, trapdoors, fences and fence gates. But I decided to cut it short in order to focus on other things with 4/13 around the corner.

@kirderf1 kirderf1 added the cleanup Patching up existing code label Apr 6, 2024
Copy link
Member

@Dweblenod Dweblenod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now thats some good stuff : ) Its clear from the generated folder that there hasnt been too many changes while switching to these helper functions so I wont worry about testing it in game.

The only thing I could think of would be to add more templates to CommonRecipes.

@kirderf1 kirderf1 merged commit cbef71a into upcoming-content Apr 9, 2024
2 checks passed
@kirderf1 kirderf1 deleted the common-recipes branch April 9, 2024 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Patching up existing code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants