Skip to content

chore(testapps): updates to dev-u-gallery and modelGarden usage#5246

Open
MichaelDoyle wants to merge 1 commit intomainfrom
md/testapps
Open

chore(testapps): updates to dev-u-gallery and modelGarden usage#5246
MichaelDoyle wants to merge 1 commit intomainfrom
md/testapps

Conversation

@MichaelDoyle
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates several core dependencies, including firebase-admin, firebase-functions, and typescript, and refactors Vertex AI Model Garden usage to use the vertexModelGarden plugin with simplified configurations. It also introduces additional evaluation metrics to the dev-ui-gallery test application and configures rootDir in multiple tsconfig.json files. Feedback identifies a potential runtime error due to misconfigured JSONATA and REGEX metrics, notes an unused dependency in the dev-ui-gallery package, and questions the validity of non-standard version numbers for cross-env and typescript.

Comment thread js/testapps/dev-ui-gallery/src/genkit.ts Outdated
Comment thread js/testapps/dev-ui-gallery/package.json Outdated
Comment thread js/testapps/dev-ui-gallery/package.json
@MichaelDoyle MichaelDoyle enabled auto-merge (squash) May 7, 2026 01:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant