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

Add CLI #4

Merged
merged 7 commits into from
Jan 28, 2024
Merged

Add CLI #4

merged 7 commits into from
Jan 28, 2024

Conversation

jayqi
Copy link
Member

@jayqi jayqi commented Jan 5, 2024

Adds a package repro-zipfile-cli that implements a CLI named reprozip repzip.

Copy link

codecov bot commented Jan 6, 2024

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

@remram44
Copy link

remram44 commented Jan 6, 2024

Note that there is already a package called reprozip which provides a command of the same name (on PyPI, Ubuntu).

Disclaimer: I wrote that reprozip.

@jayqi
Copy link
Member Author

jayqi commented Jan 6, 2024

Hi @remram44, thanks for your note. Having a program with the same name definitely seems undesirable. I'll think about an alternative.

With tests

Remove unneeded steps

Comment?

Uncomment

Add more black configuration

Fix test command

Fix mkdir test

Update release workflows

Update tag convention

Fix coverage and ruff source

Add developer notes

Fix coverage configuration

Use a counter for data for shorter file paths

Fix lint

Remove extraneous colon

Install zip for Windows

Add more tests

Use sys.executable for full path to python

Badges

Rename to repzip as a working name

Add codecov.yml configuration
@jayqi jayqi merged commit fe6275d into main Jan 28, 2024
19 checks passed
@jayqi jayqi deleted the cli branch January 28, 2024 04:30
@jayqi jayqi mentioned this pull request Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants