Skip to content

Hardening the canyonos deploy command (CAN-316) - #116

Open
Saaketh0 wants to merge 2 commits into
mainfrom
CAN-316-Harden-Deploy
Open

Saaketh0 wants to merge 2 commits into
mainfrom
CAN-316-Harden-Deploy

Conversation

@Saaketh0

@Saaketh0 Saaketh0 commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

Bulky commit, vast majority of it are tests and exception handling, there is very little new logic.
The diff is really big because lots of the code is now inside try/except blocks which adds so many new lines

New logic:

  • Fixing the homebrew packaging CI, so the brew package updates alongside PyPi on changes.
  • The canyonos sync file is much more robust (still think it needs work, so code is not the best for that command)
  • Removed canyonos version, replaced it with canyonos -v or canyonos --version
  • Made the CANYON OS banner look cooler and more polished (halftone style, with a wavy animation)
  • Removed the old ventis logo and chart that are both not used

@Saaketh0 Saaketh0 self-assigned this Sep 16, 2026
@coderabbitai

coderabbitai Bot commented Sep 16, 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: de61fe44-0f1b-47ac-ba83-7cb79b477836


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 marked this pull request as ready for review September 16, 2026 22:59
@Saaketh0 Saaketh0 changed the title Hardening the canyonos deploy command Hardening the canyonos deploy command (CAN-316) Sep 16, 2026
@Saaketh0
Saaketh0 requested a review from iidsample September 17, 2026 02:39
@Saaketh0 Saaketh0 added Non-Critical Fix Something that fixes something, but is not of utmost importance CLI Pertaining to canyonos CLI, mainly in the CLI folder labels Sep 18, 2026
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 Non-Critical Fix Something that fixes something, but is not of utmost importance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant