Skip to content

fix: clarify RC005 axis-count help - #112

Merged
vyncint merged 1 commit into
vyncint:mainfrom
Akimbo92i:fix/rc005-axis-count-help
Sep 4, 2026
Merged

fix: clarify RC005 axis-count help#112
vyncint merged 1 commit into
vyncint:mainfrom
Akimbo92i:fix/rc005-axis-count-help

Conversation

@Akimbo92i

Copy link
Copy Markdown
Contributor

Summary

  • Clarifies RC005 launch-contract mismatch help so axis counts are spelled once.
  • Avoids recommending a nonexistent three-axis index formula for domain-3 contracts.

Changes

  • Adjusted RC005 help construction for domain-2 and domain-3 launch contracts.
  • Added a domain-3 lint sample fixture.
  • Extended CLI assertions for strict text, findings JSON help, and SARIF output.
  • Added an Unreleased changelog entry for the diagnostic wording fix.

Testing

  • cargo fmt --all --check passed during implementation.
  • cargo test -p cargo-reconverge --test check_cli lint_samples_report_all_codes_and_gate_the_exit passed during implementation.
  • cargo clippy --workspace --all-targets -- -D warnings passed during implementation.
  • cargo test --workspace passed during implementation.
  • git diff --check passed during implementation and PR preparation.

Related Issue
Fixes #100

Signed-off-by: Akimbo92i <249540823+Akimbo92i@users.noreply.github.com>
@Akimbo92i
Akimbo92i marked this pull request as ready for review September 4, 2026 15:59
@Akimbo92i
Akimbo92i requested a review from vyncint as a code owner September 4, 2026 15:59
@vyncint
vyncint merged commit 6bdec17 into vyncint:main Sep 4, 2026
10 checks passed
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.

RC005's help line says "covers 2 two axes", and at domain = 3 it says "covers 3 two axes"

2 participants