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.

Sign up for GitHub

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

Jump to bottom

Support default_iff_used in ArgumentParser #22

Open
tradej opened this issue Sep 24, 2015 · 5 comments
Open

Support default_iff_used in ArgumentParser #22

tradej opened this issue Sep 24, 2015 · 5 comments
Labels
bug client

Comments

@tradej
Copy link
Member

tradej commented Sep 24, 2015

See https://github.com/devassistant/devassistant/blob/master/devassistant/cli/devassistant_argparse.py#L98
@hroncok
Copy link
Member

hroncok commented Sep 24, 2015

I think importing it from devassistant and making the client dependent on devassistant is a bad idea; so maybe just copy-paste the code?

@tradej
Copy link
Member Author

tradej commented Sep 24, 2015

Yes, copy-pasting seems the preferable solution to me too.

@hroncok
Copy link
Member

hroncok commented Sep 25, 2015

Do we need this for the proof of concept? It's not in the milestone, but I guess some assistants might need it, right?

@tradej
Copy link
Member Author

tradej commented Sep 25, 2015

We'll be demoing only the nulecule assistant, and it doesn't need it.

@hroncok
Copy link
Member

hroncok commented Sep 25, 2015 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug client
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tradej @hroncok