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
make more than one stage for all the tasks we want to do. #863
make more than one stage for all the tasks we want to do. #863
Conversation
* this should make it easier to see what failed * should allow for easier gate debugging
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.
This LGTM. Commented with potential improvements.
.travis.yml
Outdated
| - stage: "Format" | ||
| script: | ||
| - ./scripts/travis.sh format | ||
| - stage: "Vet" |
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.
Can "Format", "Vet", and "Lint" all be put under a single stage "Lint"?
.travis.yml
Outdated
| script: | ||
| - ./scripts/travis.sh build | ||
|
|
||
| - stage: "OpenShift CI" |
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.
I think this comment is outside the scope of this PR, but I was thinking it would be nice if these "OpenShift CI" and "Kubernetes CI" were separate tests belonging to the same stage "Functional Testing".
4cd9494
to
d5260ae
Compare
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.
ACK looks good
256f3b0
to
061e3fa
Compare
Describe what this PR does and why we need it: