-
Notifications
You must be signed in to change notification settings - Fork 66
✨ Add label selectors to CE API for catalog selection #1137
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.
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
Conversation
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1137 +/- ##
===========================================
- Coverage 75.44% 50.80% -24.64%
===========================================
Files 35 35
Lines 1918 1919 +1
===========================================
- Hits 1447 975 -472
- Misses 329 869 +540
+ Partials 142 75 -67
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
0c6a636
to
29e8145
Compare
go-apidiff is expected to error out with a new API change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Not sure why tilt is failing, seems to be a failure of the manager to come up? |
Signed-off-by: Todd Short <[email protected]>
New changes are detected. LGTM label has been removed. |
(Rebased to main) |
Tilt is broken until this repo is updated with the latest catalogd |
13efcd0
Fixes #1111
Description
Reviewer Checklist