Skip to content

Add --version option to CLI #496

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

Merged
merged 1 commit into from
Aug 4, 2025

Conversation

sderev
Copy link
Contributor

@sderev sderev commented Jul 23, 2025

Adds a --version option to the CLI that displays the current package version.

@sderev sderev force-pushed the feature/cli-version-option branch from 226b108 to d4aad46 Compare July 24, 2025 12:30
@coveralls
Copy link

coveralls commented Jul 24, 2025

Pull Request Test Coverage Report for Build 16523282459

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 16449711578: 0.0%
Covered Lines: 501
Relevant Lines: 501

💛 - Coveralls

@sderev sderev force-pushed the feature/cli-version-option branch 2 times, most recently from f437f53 to 92e3eac Compare July 24, 2025 16:51
@jdepoix
Copy link
Owner

jdepoix commented Jul 25, 2025

Hi @sderev!
Thank you for your contribution!
There's a bunch of unrelated formatting changes in your PR. Please remove those, because the diff get's very hard to read like this! (These shouldn't have been introduced by ruff, as long as you're using the version specified in the pyproject.toml)

@sderev sderev force-pushed the feature/cli-version-option branch from 92e3eac to 87880e7 Compare July 25, 2025 13:34
@sderev
Copy link
Contributor Author

sderev commented Jul 25, 2025

Hi @jdepoix,
Apologies for the formatting mess! Had a global ruff config interfering. I've cleaned up the PR to only include the actual feature changes. Thanks for the feedback!

Copy link
Owner

@jdepoix jdepoix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

Thank you for your contribution @sderev! 😊🙏

@jdepoix jdepoix merged commit 3e6d569 into jdepoix:master Aug 4, 2025
10 checks passed
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.

3 participants