Skip to content

fix(cli): correct Rust CLI license metadata - #3729

Open
ranxi2001 wants to merge 1 commit into
volcengine:mainfrom
ranxi2001:fix/ov-cli-license-metadata
Open

fix(cli): correct Rust CLI license metadata#3729
ranxi2001 wants to merge 1 commit into
volcengine:mainfrom
ranxi2001:fix/ov-cli-license-metadata

Conversation

@ranxi2001

Copy link
Copy Markdown

Description

Align the Rust CLI package metadata with the component's authoritative Apache-2.0 license. crates/LICENSE and the root README already identify crates/ov_cli as Apache-2.0, while its Cargo manifest incorrectly declared MIT.

Human Involvement

  • A human participated in the implementation or review loop
  • This PR was generated entirely by AI agents without human participation in the loop

Related Issue

Fixes #3689

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test update

Changes Made

  • Change the ov_cli package license expression from MIT to Apache-2.0.
  • Keep the existing Apache-2.0 license text and README license table unchanged.

Testing

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have tested this on the following platforms:
    • Linux
    • macOS
    • Windows

Validation performed:

  • Parsed crates/ov_cli/Cargo.toml with Python tomllib and asserted package.license == "Apache-2.0".
  • Confirmed crates/LICENSE contains Apache License 2.0 and the README already documents crates/ov_cli as Apache 2.0.
  • Confirmed the diff contains only crates/ov_cli/Cargo.toml and passes git diff --check.

Checklist

  • My code follows the project's coding style
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas (not applicable: no code changed)
  • I have made corresponding changes to the documentation (not applicable: existing documentation is already correct)
  • My changes generate no new warnings (Cargo validation was not available locally)
  • Any dependent changes have been merged and published (not applicable: no dependencies)

Screenshots (if applicable)

Not applicable.

Additional Notes

cargo metadata and Rust tests were not run locally because Cargo is unavailable in the current environment. This is a metadata-only, one-line SPDX correction with no runtime behavior change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

Clarify crates/ov_cli MIT vs Apache-2.0 license metadata

1 participant