Skip to content

Conversation

CorieW
Copy link
Member

@CorieW CorieW commented Aug 21, 2025

No description provided.

CorieW added 30 commits August 8, 2025 03:26
…error message when installing same version with -v (must use -r)
Copy link

@Copilot 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 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.

@pavelgj
Copy link
Collaborator

pavelgj commented Aug 29, 2025

please sync to head on main, it should fix the go build error

@pavelgj pavelgj merged commit b973d51 into main Aug 29, 2025
10 checks passed
@pavelgj pavelgj deleted the @invertase/cli-add-update-checks branch August 29, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants