Skip to content

[codex] add CLI version flag#302

Closed
cdsap wants to merge 2 commits intomainfrom
codex-add-cli-version-flag
Closed

[codex] add CLI version flag#302
cdsap wants to merge 2 commits intomainfrom
codex-add-cli-version-flag

Conversation

@cdsap
Copy link
Copy Markdown
Owner

@cdsap cdsap commented Apr 20, 2026

What changed

  • added a root --version CLI flag through Clikt
  • sourced the reported version from the project-generator library via a generated resource
  • aligned the cli module version with the library version and documented the new flag
  • added a CLI test covering --version

Why

The CLI did not expose its current version, and the version value was drifting because the library and CLI modules carried separate version definitions.

Impact

Users can now run ./projectGenerator --version and get the current library-backed release version.

Validation

  • ./gradlew :cli:test --tests io.github.cdsap.projectgenerator.cli.GenerateProjectsCliTest

@cdsap cdsap marked this pull request as ready for review April 20, 2026 02:27
Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

The job runs the full E2E suite in one step (unlike build.yml), so 60m
was occasionally exceeded.

Made-with: Cursor
Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@cdsap cdsap closed this May 4, 2026
@cdsap cdsap deleted the codex-add-cli-version-flag branch May 4, 2026 18:38
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.

1 participant