Skip to content

fix(update): run brew update before brew upgrade in upgrade hints - #98

Merged
ernestoongaro merged 2 commits into
mainfrom
fix/brew-update-before-upgrade
Sep 14, 2026
Merged

ernestoongaro merged 2 commits into
mainfrom
fix/brew-update-before-upgrade

Conversation

@ernestoongaro

Copy link
Copy Markdown
Collaborator

Summary

The Homebrew upgrade hint shown by omni update and the update notice was brew upgrade omni. brew upgrade only auto-refreshes taps once every 24h (HOMEBREW_AUTO_UPDATE_SECS), so right after a release it can report omni as already up to date. The hint is now brew update && brew upgrade omni.

brew update has no per-tap option, so this refreshes all taps.

Test plan

  • make test

🤖 Generated with Claude Code

https://claude.ai/code/session_013CdoGaVhesemjvnUd5sK8V

brew upgrade only auto-refreshes taps every 24h, so right after a release
it can report omni as up to date.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013CdoGaVhesemjvnUd5sK8V
Comment thread cmd/omni/update_notice_test.go Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can you fix these too? (clearly not your code)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Done in f1d258b — swapped all the example.test URLs for real exploreomni/cli release URLs.

… URLs

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013CdoGaVhesemjvnUd5sK8V
@ernestoongaro
ernestoongaro merged commit b542bbe into main Sep 14, 2026
2 checks passed
@ernestoongaro
ernestoongaro deleted the fix/brew-update-before-upgrade branch September 14, 2026 16:39
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