Skip to content

Conversation

@Wind292
Copy link

@Wind292 Wind292 commented Aug 11, 2025

I added a worldgen page to the docs, it includes documentation for creating trees and ores.

@Wind292 Wind292 requested a review from a team as a code owner August 11, 2025 02:03
@netlify
Copy link

netlify bot commented Aug 11, 2025

Deploy Preview for nimble-elf-d9d491 ready!

Name Link
🔨 Latest commit d423deb
🔍 Latest deploy log https://app.netlify.com/projects/nimble-elf-d9d491/deploys/693d020c0bd5c00008ce3ec3
😎 Deploy Preview https://deploy-preview-346--nimble-elf-d9d491.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@its-miroma its-miroma added new-content A new page or set of pages todo:expand This should be expanded priority:low This is nice to have, but it can wait a little labels Aug 11, 2025
Copy link
Member

@its-miroma its-miroma left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution! This is not a full review.

Please make sure to fix lint and checkstyle issues if any.

@its-miroma its-miroma added todo:verify This needs to be reviewed and removed todo:expand This should be expanded labels Aug 11, 2025
@its-miroma its-miroma linked an issue Aug 11, 2025 that may be closed by this pull request
Wind292 and others added 4 commits August 11, 2025 14:15
Co-authored-by: Miroma <136986257+its-miroma@users.noreply.github.com>
Co-authored-by: Miroma <136986257+its-miroma@users.noreply.github.com>
@its-miroma its-miroma removed the priority:low This is nice to have, but it can wait a little label Sep 17, 2025
@its-miroma its-miroma mentioned this pull request Oct 12, 2025
9 tasks
Copy link
Contributor

@skycatminepokie skycatminepokie left a comment

Choose a reason for hiding this comment

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

I don't know anything about worldgen, but this made sense to me. Very helpful. Checkstyle fixes are probably all that needs to be done.

- Argument 5 — Controls how the tree trunk tapers at different heights, primarily for larger trunks.

:::tip
We _highly_ recommend that you play around with these values to create a custom tree that **you** are happy with!
Copy link
Contributor

Choose a reason for hiding this comment

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

Can these changes be hotswapped using /reload? That might be something to mention.

@its-miroma its-miroma added the priority:medium This should be addressed relatively quickly label Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-content A new page or set of pages priority:medium This should be addressed relatively quickly todo:verify This needs to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Developer Guides: World gen (via datagen), e.g an ore

4 participants