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
Added ability to extermiante all wild creatures #999
base: master
Are you sure you want to change the base?
Conversation
@Hiranus ...and also a typo in your title; are you sure you want it out there like that? |
@@ -30,6 +30,10 @@ Examples | |||
Kill the ravens flying around the map (but only the male ones). | |||
``exterminate GOBLIN --method magma --only-visible`` | |||
Kill all visible, hostile goblins on the map by boiling them in magma. | |||
``exterminate all`` | |||
Kill all wild creatures. |
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.
Kill all wild creatures. | |
Kill all non-friendly creatures. |
``exterminate all`` | ||
Kill all wild creatures. | ||
``exterminate all:FEMALE`` | ||
Kill all wild female creatures. |
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.
Kill all wild female creatures. | |
Kill all non-friendly female creatures. |
selected_caste = tokens[2] | ||
end | ||
|
||
for _, unit in pairs(df.global.world.units.active) do |
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.
for _, unit in pairs(df.global.world.units.active) do | |
for _, unit in ipairs(df.global.world.units.active) do |
local selected_caste = nil | ||
|
||
if string.find(positionals[1], ':') then | ||
local tokens = positionals[1]:split(':') | ||
selected_caste = tokens[2] | ||
end |
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.
local selected_caste = nil | |
if string.find(positionals[1], ':') then | |
local tokens = positionals[1]:split(':') | |
selected_caste = tokens[2] | |
end | |
local selected_caste = positionals[1]:split(':')[2] |
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.
I realize this was copied from the section below, but since you don't care about the "race" (all
) here, it can be simplified
Also needs changelog entry, please |
Ping on this; have you had a chance to read the review comments? |
Added ability to exterminate all wild creatures with single command