-
Notifications
You must be signed in to change notification settings - Fork 64
🌱 (fix) remove the testdata COPY from catalogd/testdata/catalogs/test-catalog.Dockerfile to allow the docker build #1789
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
🌱 (fix) remove the testdata COPY from catalogd/testdata/catalogs/test-catalog.Dockerfile to allow the docker build #1789
Conversation
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1789 +/- ##
==========================================
- Coverage 68.37% 68.34% -0.04%
==========================================
Files 63 63
Lines 5117 5117
==========================================
- Hits 3499 3497 -2
- Misses 1389 1390 +1
- Partials 229 230 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Alternatively, for something that doesn’t need to be reverted, simply delete |
…atalog.Dockerfile to allow the docker build
c07fb0b
to
d961f8d
Compare
@@ -1,5 +1,4 @@ | |||
FROM scratch | |||
COPY test-catalog /configs |
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.
@tmshort that works as well :-P
I was silly. Good point
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
17220c2
To allow to pass: openshift/operator-framework-operator-controller#271