-
Notifications
You must be signed in to change notification settings - Fork 53
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
🌱 bump catalogd to v0.30.0 #1329
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 #1329 +/- ##
===========================================
- Coverage 76.37% 53.65% -22.73%
===========================================
Files 41 41
Lines 2438 2438
===========================================
- Hits 1862 1308 -554
- Misses 402 1027 +625
+ Partials 174 103 -71
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
81576f5
to
4679c37
Compare
@joelanford This PR needs rebase. |
4679c37
to
1ac3c3c
Compare
After this merges, we'll need to change |
Signed-off-by: Joe Lanford <[email protected]>
1ac3c3c
to
e1f7cfc
Compare
The upgrade-e2e is expected to fail here. We made breaking changes in catalogd, so the failure is a true positive. This error is happening because the I think this PR is ready to merge. |
e57525e
Description
Reviewer Checklist