-
Notifications
You must be signed in to change notification settings - Fork 64
🐛 renamed catalogd service name in download-catalog hack script #1545
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
🐛 renamed catalogd service name in download-catalog hack script #1545
Conversation
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
👍 I could not find any place that is calling the value fixed (indeed downstream)
So, it seems fine to me.
/lgtm
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1545 +/- ##
=======================================
Coverage 74.25% 74.25%
=======================================
Files 42 42
Lines 3329 3329
=======================================
Hits 2472 2472
Misses 676 676
Partials 181 181
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
83b01c3
…perator-framework#1545) Co-authored-by: Artur Zych <[email protected]>
…perator-framework#1545) Co-authored-by: Artur Zych <[email protected]>
Small fix that changes the name of the catalogd service from
catalogd-server
to the currently usedcatalogd-service
in the hack/tools/catalogs/download-catalog script.Changes separated from #1539 as per #1539 (comment)
Reviewer Checklist