Skip to content

Simplify self-update command from 'openboot update --self' to 'openboot update'#36

Merged
fullstackjam merged 1 commit intomainfrom
claude/fix-upgrade-notification-alignment-ZBbyX
Apr 19, 2026
Merged

Simplify self-update command from 'openboot update --self' to 'openboot update'#36
fullstackjam merged 1 commit intomainfrom
claude/fix-upgrade-notification-alignment-ZBbyX

Conversation

@fullstackjam
Copy link
Copy Markdown
Collaborator

What does this PR do?

Updates user-facing messages to reflect the simplified self-update command syntax, removing the --self flag from upgrade instructions.

Why?

The --self flag appears to be redundant or has been removed from the command interface. This change simplifies the user experience by providing the correct, shorter command syntax in help messages and error notifications.

Testing

  • go vet ./... passes
  • Tested locally by triggering update notifications and upgrade failures to verify correct messages are displayed

Notes for reviewer

This is a straightforward documentation/messaging update affecting two user-facing strings in the updater module. No functional logic changes are involved.

https://claude.ai/code/session_0149hb3rTyU8APZ8TNRUbBvg

The notify-only auto-update path told users to run 'openboot update
--self', but no --self flag exists on the update command. Use the
real invocation: 'openboot update'.
@github-actions
Copy link
Copy Markdown

👋 Thanks for opening this pull request!

Before merging:

  • Code follows existing patterns in the codebase
  • go build ./... and go vet ./... pass
  • Commit message is clear and descriptive

@fullstackjam will review this soon. Thanks for contributing! 🚀

@fullstackjam fullstackjam merged commit f892a1a into main Apr 19, 2026
9 checks passed
@fullstackjam fullstackjam deleted the claude/fix-upgrade-notification-alignment-ZBbyX branch April 19, 2026 13:56
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 19, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/updater/updater.go 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

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