-
Notifications
You must be signed in to change notification settings - Fork 204
fix dockefile path #4721
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 dockefile path #4721
Conversation
Signed-off-by: hungran <[email protected]>
khanhtc1202
left a comment
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.
👍👍
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4721 +/- ##
==========================================
+ Coverage 30.83% 30.84% +0.01%
==========================================
Files 221 221
Lines 25993 25993
==========================================
+ Hits 8014 8018 +4
+ Misses 17329 17325 -4
Partials 650 650 ☔ View full report in Codecov by Sentry. |
ffjlabo
left a comment
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.
Thank you !🚀 🚀
Signed-off-by: hungran <[email protected]>
Signed-off-by: hungran <[email protected]>
Signed-off-by: hungran <[email protected]> Signed-off-by: sZma5a <[email protected]>
Signed-off-by: hungran <[email protected]> Signed-off-by: sZma5a <[email protected]>
Signed-off-by: hungran <[email protected]> Signed-off-by: 鈴木 優耀 <[email protected]>
What this PR does / why we need it: fix the last PR
#4719
#4718
#4712
seem like hard to maintain the image because of fragment on base tool image and the 2nd iterator.
even better by multistage build tho?
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: n/a