-
Notifications
You must be signed in to change notification settings - Fork 13
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
rename bundle containerfile to Containerfile.bundle to be consistent #378
rename bundle containerfile to Containerfile.bundle to be consistent #378
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #378 +/- ##
=======================================
Coverage 28.13% 28.13%
=======================================
Files 128 128
Lines 11209 11209
=======================================
Hits 3154 3154
Misses 7773 7773
Partials 282 282
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
353065a
to
dd68077
Compare
…n the repo Signed-off-by: Mohamed Mahmoud <[email protected]>
dd68077
to
6a69083
Compare
Do you need to update Container.bundle.openshift with sdk 1.27.0? |
probably at some point but not urgent |
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
bpfman#378 renamed bundle.Dockerfile as Containerfile.bundle to be consistent with other Containerfiles in the repository. However, bundle.Dockerfile is an autogenerate file that is generated during the `make bundle` command. So revert the rename and leave it as it was. Signed-off-by: Billy McFall <[email protected]>
rename bundle container file to be consistent with other container files in the repo