Skip to content

feat(cli): replace canyonos version with -v/--version - #188

Merged
userAugustos merged 1 commit into
mainfrom
feat/cli-version
Sep 23, 2026
Merged

userAugustos merged 1 commit into
mainfrom
feat/cli-version

Conversation

@Saaketh0

Copy link
Copy Markdown
Collaborator

Removes the old canyonos version command in favor of canyonos -v / canyonos --version. The flags already exist in cli.py, so this PR only cleans up the leftover references:

  • README command table now lists -v, --version instead of version
  • Homebrew formula test runs canyonos --version, since canyonos version no longer exists and would fail
  • CLI module docstring no longer mentions a version command

🤖 Generated with Claude Code

Drop the remaining references to the old version subcommand from the
README, the Homebrew formula test, and the CLI module docstring.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 23, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 4aadfadf-b3cc-498f-9eb3-a961e4bdb970


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Saaketh0 Saaketh0 added Quality of Life Something being made easier, for anyone of notice CLI Pertaining to canyonos CLI, mainly in the CLI folder labels Sep 23, 2026
@Saaketh0 Saaketh0 self-assigned this Sep 23, 2026

@nickhuo nickhuo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

for the cli version, is it good to set up a auto upgrade before the deploy or test?

good move, lgtm

@Saaketh0

Copy link
Copy Markdown
Collaborator Author

@nickhuo eh I think we'll just alert them to upgrade instead of doing it for them, that is a different PR

@Saaketh0
Saaketh0 enabled auto-merge (squash) September 23, 2026 18:49

@userAugustos userAugustos left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@userAugustos
userAugustos merged commit a872c93 into main Sep 23, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLI Pertaining to canyonos CLI, mainly in the CLI folder Quality of Life Something being made easier, for anyone of notice

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants