-
Notifications
You must be signed in to change notification settings - Fork 199
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
Burial update #859
Burial update #859
Conversation
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
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.
be sure to also add a note to changelog.txt
to document that this tool has been updated and reinstated
``burial -cp`` | ||
Create tombs with automatic burial disabled for both citizens and pets, | ||
requiring manual assignment of deceased units to each tomb. |
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.
nice. I like that you documented this use case
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.
Thank you. Two options logically excluding each other probably isn't the most semantically straightforward way to access a feature, but it puts the docs to good use
Co-authored-by: Myk <[email protected]>
Reworked the burial script to create minimal tombs for each coffin building, allowing things to be buried in them.
Also modified the options to allow for designating dwarf-only, pet-only, or general purpose tombs.