-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fixes issue #391: Replaced calling docker directly by using "make build-docker" #522
Fixes issue #391: Replaced calling docker directly by using "make build-docker" #522
Conversation
Codecov Report
@@ Coverage Diff @@
## master #522 +/- ##
======================================
Coverage 65.8% 65.8%
======================================
Files 11 11
Lines 1000 1000
======================================
Hits 658 658
Misses 305 305
Partials 37 37 Continue to review full report at Codecov.
|
@@ -15,7 +15,7 @@ before_install: | |||
|
|||
script: | |||
- GO111MODULE=on go test ./... | |||
- docker build --tag kube-bench . | |||
- IMAGE_NAME=kube-bench make build-docker |
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.
Is there any reason to keep IMAGE_NAME var outside of makefile?
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.
The use of the IMAGE_NAME is to make sure the line following that one continues to work.
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.
All right, I thought that the defaults hardcoded in makefile are enough, so when I run make build-docker
it would work the same way as on Travis.
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
Fixes issue #391:
Replaced calling docker directly by using "make build-docker"