-
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
ci: Run Containerfile, Builder Containerfile #951
Conversation
Benchmark ResultsSummary
✅ See Better Results...
❌ See Worse Results...
✨ See Unchanged Results...
🐋 See Full Results...
|
2f2bc6e
to
2a77bd6
Compare
2a77bd6
to
66a3f91
Compare
Just noting that I am happy with it, but am highly inexperienced with docker and would like you to wait for approval from someone who is not me before merging. |
two criticism of this PR are : my replies to this: |
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.
A couple comments to discuss before approval :)
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
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 (I didn't test the file, assuming you have ran it and tested it).
Codecov Report
@@ Coverage Diff @@
## develop #951 +/- ##
===========================================
+ Coverage 68.14% 68.16% +0.01%
===========================================
Files 172 172
Lines 16246 16246
===========================================
+ Hits 11071 11074 +3
+ Misses 4245 4243 -2
+ Partials 930 929 -1
|
Relevant issue(s)
Resolves #950
Description
Tasks
How has this been tested?
Specify the platform(s) on which this was tested: