Skip to content
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

feat(*): Github Actions for PR builds #643

Merged
merged 2 commits into from
Dec 19, 2019
Merged

Conversation

ajordens
Copy link
Contributor

No description provided.

@fieldju
Copy link
Collaborator

fieldju commented Dec 18, 2019

Would still like the functional tests to run in CI
https://github.com/spinnaker/kayenta/blob/master/.travis.yml#L14

@ajordens
Copy link
Contributor Author

ajordens commented Dec 18, 2019

Oh interesting ... that's non-standard but I'll add the same env variable.

Thanks for the heads up @fieldju!

@fieldju
Copy link
Collaborator

fieldju commented Dec 18, 2019

Does this PR trigger the action or does it have to be merged and then it works with future PRs?
I was going to try to look at the logs and make sure all that docker stuff for the functional tests still works in the new CI env.

@ajordens
Copy link
Contributor Author

I think it'll have to merge first and then it'll run on future PRs.

This isn't taking away TravisCI builds (and we'll still be relying on a 👍 from it).

@ajordens ajordens merged commit 7a7bf14 into spinnaker:master Dec 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants