Skip to content

create parity between //snow and //thaw#2940

Merged
octylFractal merged 4 commits intoEngineHub:masterfrom
SamuraiWorf:fix/thaw-parity
Apr 20, 2026
Merged

create parity between //snow and //thaw#2940
octylFractal merged 4 commits intoEngineHub:masterfrom
SamuraiWorf:fix/thaw-parity

Conversation

@SamuraiWorf
Copy link
Copy Markdown
Contributor

makes //thaw and //snow cover the same area

@SamuraiWorf SamuraiWorf requested a review from a team as a code owner April 6, 2026 17:48
Copy link
Copy Markdown
Member

@octylFractal octylFractal left a comment

Choose a reason for hiding this comment

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

Is it possible to also make thaw use block positions from a CylinderRegion instead? I'm unsure if it works properly, but I would rather use the same source of truth if we can.

@SamuraiWorf
Copy link
Copy Markdown
Contributor Author

Done as suggested

Copy link
Copy Markdown
Member

@octylFractal octylFractal left a comment

Choose a reason for hiding this comment

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

BTW, if you're able to, re-requesting a review helps me sort PRs better.

Comment thread worldedit-core/src/main/java/com/sk89q/worldedit/EditSession.java Outdated
@octylFractal
Copy link
Copy Markdown
Member

Since this is feature work, I'm going to put it in master.

@octylFractal octylFractal changed the base branch from version/7.4.x to master April 20, 2026 09:11
@octylFractal octylFractal added this pull request to the merge queue Apr 20, 2026
@octylFractal octylFractal added this to the 8.0.0 milestone Apr 20, 2026
Merged via the queue into EngineHub:master with commit ed4a31a Apr 20, 2026
2 checks passed
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