-
Notifications
You must be signed in to change notification settings - Fork 368
feat(cli): add update checks #3464
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…tase/cli-add-update-cmd
…al install, inquire it
…error message when installing same version with -v (must use -r)
…rebase/genkit into @invertase/cli-add-update-cmd
There was a problem hiding this 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 introduces update checking functionality to the Genkit CLI, allowing users to be notified when newer versions are available. The feature includes comprehensive configuration options and platform-aware update instructions.
- Adds update notification system with support for both npm and binary installations
- Implements user configuration to opt-out of update notifications
- Provides platform-specific update instructions for different installation methods
Reviewed Changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
genkit-tools/cli/src/utils/updates.ts | Core update checking logic with npm registry and GCS integration |
genkit-tools/cli/tests/utils/updates_test.ts | Comprehensive test suite covering all update functionality scenarios |
genkit-tools/cli/src/commands/config.ts | Adds configuration option for disabling update notifications |
genkit-tools/cli/src/cli.ts | Integrates update checking into CLI startup flow |
genkit-tools/cli/package.json | Updates dependencies and build configuration for version handling |
Files not reviewed (1)
- genkit-tools/pnpm-lock.yaml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
please sync to head on main, it should fix the go build error |
No description provided.