Skip to content

Conversation

@ronakgh97
Copy link

Four small fixes:

  1. omarchy-theme-set — Logic error: [[ -z $1 && $1 != "CNCLD" ]] always evaluates wrong. Changed to [[ -z $1 || $1 == "CNCLD" ]]
  2. omarchy-install-docker-dbs — Calls undefined main_menu on Escape. Changed to exit 0
  3. omarchy-cmd-screenshot — Little typo: "selction" → "selection"
  4. omarchy-upload-log — Usage says system-info but case uses installed

ryanrhughes and others added 2 commits November 28, 2025 20:13
- omarchy-theme-set: Fix logic error (&&/!= should be ||/==)
- omarchy-install-docker-dbs: Replace undefined main_menu with exit 0
- omarchy-cmd-screenshot: Fix little typo here 'selction' -> 'selection'
- omarchy-upload-log: Fix usage message as the message suggested (system-info -> installed)
@ronakgh97 ronakgh97 changed the title Fix argument handling bugs and typos Fix argument bugs Nov 30, 2025
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.

2 participants