Skip to content

Conversation

@dordsor21
Copy link
Member

@dordsor21 dordsor21 requested a review from a team as a code owner August 1, 2025 13:52
@github-actions github-actions bot added the Bugfix This PR fixes a bug label Aug 1, 2025
@PierreSchwang
Copy link
Member

Given you updated the list of ores: is / was there any plan to add the deepslate variants as well?

@dordsor21
Copy link
Member Author

Given you updated the list of ores: is / was there any plan to add the deepslate variants as well?

Mm yeah the ore generation settings aren't too obfuscated in the code so wouldn't be impossible


public abstract class FaweRegionExtent extends ResettableExtent implements IBatchProcessor {

private final FaweLimit limit;
Copy link
Member

Choose a reason for hiding this comment

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

Can we mark this and the ctor param as nullable?

@dordsor21 dordsor21 requested review from a team and SirYwell October 30, 2025 21:15
@dordsor21 dordsor21 requested review from a team and PierreSchwang October 30, 2025 22:10
 - also add copper and update limits
 - also restrict ores spawning to region
 - also fix caves out of region error
 - fixes #3108
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bugfix This PR fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

//caves Permission Issue //Ores Does Not //undo Consistently

4 participants