fix: add version command to Specify CLI #811
#869
Conversation
lshgdut
changed the title
version command to Specify CLI #811
There was a problem hiding this comment.
Pull Request Overview
This PR adds version display functionality to the Specify CLI by implementing both a version subcommand and a --version flag for the root command. The changes utilize Python's importlib.metadata to dynamically retrieve the package version.
- Added
versionsubcommand that displays the application version - Added
--versionflag to the root command callback for immediate version display - Integrated
importlib.metadatato dynamically fetch package version information
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| import ssl | ||
| import truststore | ||
|
|
||
| __version__ = importlib.metadata.version(__package__) |
There was a problem hiding this comment.
This line will fail if __package__ is None (which happens when the module is run as a script). Consider using a fallback like importlib.metadata.version('specify_cli') or adding error handling.
| __version__ = importlib.metadata.version(__package__) | |
| __version__ = importlib.metadata.version(__package__ or "specify_cli") |
Copilot uses AI. Check for mistakes.
|
|
||
| def show_version(): | ||
| """Show the application version.""" | ||
| typer.echo(f"{__package__} {__version__}") |
There was a problem hiding this comment.
This will output 'None' for the package name if __package__ is None. Consider using a hardcoded package name like 'specify_cli' or the actual package name constant.
Copilot uses AI. Check for mistakes.
1 participant
Add features to Specify CLI:
versionsub command--versionflag for root commandClosed issues: #486 #811