Skip to content

dart evals docs#262

Open
pavelgj wants to merge 2 commits intomainfrom
pj/dart-evals
Open

dart evals docs#262
pavelgj wants to merge 2 commits intomainfrom
pj/dart-evals

Conversation

@pavelgj
Copy link
Copy Markdown
Member

@pavelgj pavelgj commented Apr 16, 2026

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 adds Dart support to the Genkit evaluation documentation and refactors the document to share common introductory text across JavaScript, Go, Python, and Dart. A review comment points out a syntax error in the Dart code snippet where schema definitions are missing the 'Schema' class prefix.

Comment thread src/content/docs/docs/evaluation.mdx
@abdallahshaban557
Copy link
Copy Markdown
Collaborator

/gcbrun

@chrisraygill
Copy link
Copy Markdown
Collaborator

@pavelgj I think this might have at least 1 error. Genkit Dart doesn't have a defineRetriever right?

@pavelgj
Copy link
Copy Markdown
Member Author

pavelgj commented Apr 20, 2026

@pavelgj I think this might have at least 1 error. Genkit Dart doesn't have a defineRetriever right?

ah... forgot to add "no mistakes"... should be good now.

@pavelgj pavelgj requested a review from chrisraygill April 20, 2026 15:19
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.

3 participants