Skip to content

Conversation

@ryanfowler
Copy link
Owner

No description provided.

@ryanfowler ryanfowler requested a review from Copilot January 30, 2026 23:04
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds --dry-run support for the --update flag, allowing users to check for available updates without installing them.

Changes:

  • Added dryRun parameter to update-related functions to propagate the dry-run flag through the update flow
  • Implemented dry-run logic to display update availability without performing the actual update
  • Added integration tests to verify dry-run behavior when already on latest version and when updates are available
  • Updated CLI documentation to describe the new --dry-run behavior with --update

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
main.go Passes the app.DryRun flag to the update.Update function
internal/update/update.go Adds dryRun parameter to update functions and implements early return logic when dry-run is enabled
integration/integration_test.go Adds test cases to verify dry-run update behavior for both same version and available update scenarios
docs/cli-reference.md Updates documentation for --update and --dry-run flags to describe the new combined behavior

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ryanfowler ryanfowler enabled auto-merge January 30, 2026 23:05
@ryanfowler ryanfowler merged commit 8fe13e5 into main Jan 30, 2026
10 checks passed
@ryanfowler ryanfowler deleted the dry-run-update branch January 30, 2026 23:07
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.

1 participant