Skip to content

Conversation

unboundlopez
Copy link

@unboundlopez unboundlopez commented Aug 8, 2025

Please update exterminate to add a -c flag that targets only child units. I plan to build a timer-based script to run this so poults don’t wander off and put dwarven children at risk.

Argument parsing: adds -c alongside existing flags.
-c purpose: target children only.

Example:
exterminate -f -c bird_turkey -m knockout

[edit Aug/09/2025]
I do not know exactly how to successfully add a changelog.txt so I'm going to add it here in the description of what I believe needs to bee added in this changelog.txt.

  • exterminate: now accepts a "-c" argument which targets only children. Case sensitive.

@unboundlopez
Copy link
Author

pre-commit.ci autofix

@ab9rf
Copy link
Member

ab9rf commented Aug 8, 2025

needs changelog

@unboundlopez
Copy link
Author

needs changelog

Thanks I’ll do this in about eight hours once I get out of work.

@unboundlopez
Copy link
Author

unboundlopez commented Aug 9, 2025

I do not know exactly how to successfully add a changelog.txt so I'm going to add it here in the description of what I believe needs to bee added in this changelog.txt.

exterminate: now accepts a "-c" argument which targets only children. Case sensitive.

{'m', 'method', handler = function(arg) options.method = killMethod[arg:upper()] end, hasArg = true},
{'o', 'only-visible', handler = function() options.only_visible = true end},
{'f', 'include-friendly', handler = function() options.include_friendly = true end},
{'c', 'children-only', handler = function() options.children_only = true end},
Copy link
Member

Choose a reason for hiding this comment

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

I would prefer that if you're going to add "children-only" you should also add "adult-only" just for parallelism

also, this won't kill babies. is this intended? should it kill babies? if not, should it offer a "baby-only" option?

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