Skip to content
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

Move fastdwarf/teledwarf to separate commands #665

Open
lethosor opened this issue Aug 6, 2015 · 2 comments
Open

Move fastdwarf/teledwarf to separate commands #665

lethosor opened this issue Aug 6, 2015 · 2 comments
Labels
idea Suggestions, etc.

Comments

@lethosor
Copy link
Member

lethosor commented Aug 6, 2015

There's no need to move teledwarf to a separate plugin, but a separate command could be less confusing and would allow teledwarf to be toggled without figuring out what the current fastdwarf state is and specifying it as well.
@expwnent looks like you added teledwarf originally - what do you think?

@lethosor lethosor added the idea Suggestions, etc. label Aug 6, 2015
@PeridexisErrant
Copy link
Contributor

Sounds good to me.

The superdwarf script is equivalent to (optionally) per-dwarf fastdwarf, which is probably pretty slow (since it iterates every tick). That function should probably be built into the plugin instead.

@expwnent
Copy link
Member

Yeah, I approve.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idea Suggestions, etc.
Projects
None yet
Development

No branches or pull requests

3 participants