Skip to content

Use package versions in CLI metadata#488

Merged
k82cn merged 3 commits into
xflops:mainfrom
k82cn:codex/cli-version-package-main
Jun 1, 2026
Merged

Use package versions in CLI metadata#488
k82cn merged 3 commits into
xflops:mainfrom
k82cn:codex/cli-version-package-main

Conversation

@k82cn
Copy link
Copy Markdown
Contributor

@k82cn k82cn commented Jun 1, 2026

Summary

  • replace hardcoded Clap 0.5.0 versions with package-derived versions for CLI binaries
  • standardize affected CLI author metadata to XFLOPS support@xflops.io
  • move Rust SDK/stdng publish metadata out of the release-process docs branch

Verification

  • cargo fmt --check
  • cargo check -p stdng
  • cargo check -p flame-rs --features macros
  • cargo build -p flame-session-manager -p flame-executor-manager -p flmctl -p flmping -p flmexec -p flame-object-cache
  • --version smoke checks for all six CLI binaries

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request standardizes CLI metadata across several components by updating the author to 'XFLOPS support@xflops.io' and using the #[command(version)] attribute to dynamically retrieve the version. It also bumps the stdng crate version to 0.1.8, adds package metadata to its Cargo.toml, and specifies explicit versions for path dependencies in sdk/rust/Cargo.toml. There are no review comments, and I have no feedback to provide.

@k82cn k82cn marked this pull request as ready for review June 1, 2026 07:21
@k82cn k82cn force-pushed the codex/cli-version-package-main branch from 5b63c3b to 7275319 Compare June 1, 2026 07:45
@k82cn k82cn merged commit ec061a2 into xflops:main Jun 1, 2026
8 checks passed
@k82cn k82cn deleted the codex/cli-version-package-main branch June 1, 2026 08:02
k82cn added a commit that referenced this pull request Jun 1, 2026
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