Skip to content

feat: add --version flag to CLI (fixes #63) - #65

Merged
parneetsingh022 merged 2 commits into
parneetsingh022:mainfrom
Dreamstick9:feat/add-version-flag
Dec 19, 2025
Merged

feat: add --version flag to CLI (fixes #63)#65
parneetsingh022 merged 2 commits into
parneetsingh022:mainfrom
Dreamstick9:feat/add-version-flag

Conversation

@Dreamstick9

@Dreamstick9 Dreamstick9 commented Dec 19, 2025

Copy link
Copy Markdown
Contributor

Description

Implements the requested --version flag for the CLI.

Updated src/pygha/cli.py to include the argument using action="version".

Added a new test case in tests/test_cli.py to verify the correct version string is printed and the system exits with code 0.

Related Issue

Fixes #63

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist

  • All tests pass locally (pytest)
  • I have written or updated tests for any changes
  • I have updated the documentation for any changes
  • I have updated changelog.md and referred to the issue in the changelog entry

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@parneetsingh022

Copy link
Copy Markdown
Owner

Hi @Dreamstick9, thanks for your PR, it looks good. Could you please update docs/cli.rst and CHANGELOG.md as well? Once those are updated, I’ll be able to merge your PR.

@github-actions

Copy link
Copy Markdown

Hi @Dreamstick9, this is the PyGHA bot! 🤖

Thanks for your contribution! We will review your changes and get back to you soon.

All standard checks (Tests, Docs, Changelog) look good! ✅

Please make sure you've read our Contributing Guidelines.

By contributing, you agree that your contributions will be licensed under the MIT License.

@parneetsingh022

Copy link
Copy Markdown
Owner

@Dreamstick9 Thanks for the PR. All changes looks good, I'm merging the PR.

@parneetsingh022
parneetsingh022 merged commit a934861 into parneetsingh022:main Dec 19, 2025
13 checks passed
parneetsingh022 added a commit that referenced this pull request Dec 19, 2025
parneetsingh022 added a commit that referenced this pull request Dec 19, 2025
@parneetsingh022 parneetsingh022 mentioned this pull request Dec 19, 2025
8 tasks
parneetsingh022 added a commit that referenced this pull request Dec 19, 2025
Correct link reference for issue #65 in changelog
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.

Add --version flag to the CLI

3 participants