Skip to content

CLI: click -> argparse #400

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
Merged
merged 8 commits into from
Oct 2, 2022
Merged

CLI: click -> argparse #400

merged 8 commits into from
Oct 2, 2022

Conversation

@tony
Copy link
Member

@tony tony commented Oct 2, 2022 •
edited
Loading

  • Remove click and sphinx-click
  • Add sphinx-argparse
  • Use argparse
  • Remove completion, for now

@tony tony force-pushed the vanilla-argparse branch from 098189c to eb67a30 Compare October 2, 2022 14:09
@codecov
Copy link

codecov bot commented Oct 2, 2022 •
edited
Loading

Codecov Report

Merging #400 (11a3a01) into master (11a3a01) will not change coverage.
The diff coverage is n/a.

❗ Current head 11a3a01 differs from pull request most recent head 9f36207. Consider uploading reports for the commit 9f36207 to get more accurate results

@@           Coverage Diff           @@
##           master     #400   +/-   ##
=======================================
  Coverage   84.59%   84.59%           
=======================================
  Files          15       15           
  Lines         844      844           
  Branches      133      133           
=======================================
  Hits          714      714           
  Misses         94       94           
  Partials       36       36

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tony tony force-pushed the vanilla-argparse branch from fe2ff60 to 99aac7b Compare October 2, 2022 14:27
@tony tony mentioned this pull request Oct 2, 2022
Open
1 task
@tony tony marked this pull request as ready for review October 2, 2022 14:29
@tony tony force-pushed the vanilla-argparse branch from 0fd2a43 to 9f36207 Compare October 2, 2022 14:38
@tony tony merged commit 2b012f5 into master Oct 2, 2022
@tony tony deleted the vanilla-argparse branch October 2, 2022 14:43
tony added a commit that referenced this pull request Oct 2, 2022
@tony
Tag v1.15.0a0 (click to argparse, #400)
047c2d2
tony added a commit that referenced this pull request Oct 2, 2022
@tony
Tag v1.15.0 (remove click dependency in favor of argparse, #400)
b822822
@tony tony mentioned this pull request Oct 2, 2022
Merged
@tony tony changed the title Click -> argparse CLI: click -> argparse Oct 2, 2022
@tony tony mentioned this pull request Oct 2, 2022
Merged
tony added a commit to tmux-python/tmuxp that referenced this pull request Oct 9, 2022
@tony
CLI: click -> argparse (#830)
9813ce2 
- Remove `click` and `sphinx-click`
- Add `sphinx-argparse`
- Use [`argparse`](https://docs.python.org/3/library/argparse.html)
- Remove completion, for now

See also: vcs-python/vcspull#400
@tony tony mentioned this pull request Oct 16, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tony