-
Notifications
You must be signed in to change notification settings - Fork 886
[WIP] migration to circleci 2.0 #2107
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
Conversation
da3f3b7 to
a0ffe28
Compare
e8bbc7c to
3c80f06
Compare
This commit will allow us to use newer version of CircleCI. Signed-off-by: selansen <elango.siva@docker.com>
0f56c3f to
d0f3fd8
Compare
67d64b5 to
2d9637a
Compare
Signed-off-by: Flavio Crisciani <flavio.crisciani@docker.com>
| pushd . &> /dev/null ; \ | ||
| cd $$dir ; \ | ||
| go test ${INSIDECONTAINER} -test.parallel 5 -test.v -covermode=count -coverprofile=./profile.tmp ; \ | ||
| go test ${INSIDECONTAINER} -test.v test.parallel 5 -covermode=count -coverprofile=./profile.tmp ; \ |
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.
missing the - to parallel
|
|
||
| integration-tests: ./bin/dnet | ||
| @./test/integration/dnet/run-integration-tests.sh | ||
| integration-tests: |
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.
going away
| github.com/gordonklaus/ineffassign \ | ||
| github.com/client9/misspell/cmd/misspell \ | ||
| honnef.co/go/tools/cmd/gosimple | ||
|
|
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.
remove
|
Closing in favor of: #2168 |
Signed-off-by: Flavio Crisciani flavio.crisciani@docker.com