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

Parse arguments using docopt #75

Closed
wants to merge 2 commits into from
Closed

Parse arguments using docopt #75

wants to merge 2 commits into from

Conversation

Nurdok
Copy link
Member

@Nurdok Nurdok commented Jul 1, 2014

No description provided.

@sigmavirus24
Copy link
Member

I wonder if using docopt will limit (and potentially break) the flake8-docstrings extension. We don't currently muck with the options provided by pep257 but we will not that the tool provides a --count flag.

@keleshev
Copy link
Contributor

keleshev commented Jan 9, 2015

Believe it or not, I'm not a big fan of this change either. I liked having no dependencies.

@Nurdok
Copy link
Member Author

Nurdok commented Jan 9, 2015

@sigmavirus24, I didn't quite understand what you're saying. In what way will flake8-docstrings will be limited by this change?
@keleshev, I know, I'm hesitant about adding dependencies as well and that's why I haven't merged this PR yet. Although I'm not sure if having no dependencies is such a big advantage anyway - I certainly wouldn't mind this when using pep257 in my projects.

@Nurdok Nurdok closed this Oct 20, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants