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

mock common subscription data for local eval #36

Merged

rnorton5432 merged 1 commit into main from local-mock-common-subscription-data

Jan 10, 2024

Contributor

rnorton5432 commented Jan 9, 2024

It is expected that a CommonSubscriptionQueryResult is always available, so we should mock it to a dummy value as goal-evaluation-skill does.


          mock common subscription data for local eval

444212c

rnorton5432 requested review from jgdavey, chrispatrick and felipecruz91

January 9, 2024 22:50

chrispatrick approved these changes

View reviewed changes

rnorton5432 merged commit cf01e6f into main

2 checks passed

cdupuis deleted the local-mock-common-subscription-data branch

February 2, 2024 16:47

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet