Skip to content
This repository has been archived by the owner on Sep 23, 2023. It is now read-only.

Make --force-remove-failed-dirs enabled by default? #26

Open
jairideout opened this issue Mar 22, 2013 · 3 comments
Open

Make --force-remove-failed-dirs enabled by default? #26

jairideout opened this issue Mar 22, 2013 · 3 comments
Labels

Comments

@jairideout
Copy link
Member

Should failed deploy directories be removed by default? In our documentation (and whenever I use qiime-deploy) I always supply this option. It'd make the standard qiime-deploy commands a bit shorter.

If we went this route, we could remove --force-remove-failed-dirs and add a new option (disabled by default) such as --keep-failed-dirs.

@pdmars
Copy link
Contributor

pdmars commented Mar 22, 2013

Hmm... I'm always hesitant to remove anything without forcing the user to say "yes, I want to remove that."

@jairideout
Copy link
Member Author

I agree, but at the same time, these are directories that qiime-deploy is creating during the deploy process (and that are useless except for debugging where a build failed).

@pdmars
Copy link
Contributor

pdmars commented Mar 22, 2013

Yeah, that's true. I think I could be convinced if people are comfortable with that being the default behavior.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants