Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Increase code coverage to 90% #633

Merged
merged 7 commits into from
Jun 25, 2024
Merged

Increase code coverage to 90% #633

merged 7 commits into from
Jun 25, 2024

Conversation

ntlind
Copy link
Contributor

@ntlind ntlind commented Jun 25, 2024

No description provided.

@ntlind ntlind self-assigned this Jun 25, 2024
@ntlind ntlind changed the title # Add LLMClient tests Increase code coverage to 90% Jun 25, 2024
@ntlind ntlind marked this pull request as ready for review June 25, 2024 17:24
@ntlind ntlind added the testing label Jun 25, 2024
bnativi
bnativi approved these changes Jun 25, 2024
View reviewed changes
Copy link
Contributor

@bnativi bnativi left a comment

Choose a reason for hiding this comment

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

lgtm

@ntlind ntlind merged commit faee691 into add_llm_guided_metrics Jun 25, 2024
11 checks passed
@ntlind ntlind deleted the add_llm_tests branch June 25, 2024 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bnativi bnativi bnativi approved these changes

@czaloom czaloom Awaiting requested review from czaloom

@ekorman ekorman Awaiting requested review from ekorman

Assignees

@ntlind ntlind

Labels
testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ntlind @bnativi