-
Notifications
You must be signed in to change notification settings - Fork 65
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
Structure Block Registry Processor #626
Structure Block Registry Processor #626
Conversation
… block registry internally. In the middle of testing issues with any/all processors in the template pool of the testing structure
- Cleaned up code
src/main/resources/data/minestuck/worldgen/processor_list/structure_block_registry.json
Outdated
Show resolved
Hide resolved
src/main/java/com/mraof/minestuck/world/gen/feature/StructureBlockRegistryProcessor.java
Show resolved
Hide resolved
src/main/java/com/mraof/minestuck/world/gen/feature/StructureBlockRegistryProcessor.java
Outdated
Show resolved
Hide resolved
- Made StructureBlockRegistryProcessor final
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks all good. Do you think there's a point in noting this as a datapack feature in the changelog?
With everything else fine, the test structure ought to be removed. |
- Removed test structure
Removed the structure files but did keep the (unused) processor_list under the assumption it can be used in the near future |
Made changes to StructureBlockRegistryProcessor which allows StructureBlockRegistry to be called within finalizeProcess(). This makes it viable for use directly in a
template_pool
, meaning the structure block registry is suitable for jigsaw structures!An example structure has been included for review purposes. It will need to be removed before merging.