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

ci(cirrus): use cirrus-ci alongisde travis-ci #11

Merged

Conversation

myii
Copy link
Member

@myii myii commented May 28, 2019

@myii myii force-pushed the ci/cirrus-alongside-travis branch 2 times, most recently from 64b23fb to e4b55b6 Compare May 28, 2019 20:01
@myii myii changed the title ci(cirrus): use cirrus-ci alongisde travis-ci WIP: ci(cirrus): use cirrus-ci alongisde travis-ci May 28, 2019
@myii
Copy link
Member Author

myii commented May 28, 2019

OK, we've got to wait until Cirrus CI is enabled for this repo...

@myii myii force-pushed the ci/cirrus-alongside-travis branch 2 times, most recently from 5b9320a to 00f61e7 Compare May 28, 2019 20:34
@myii myii changed the title WIP: ci(cirrus): use cirrus-ci alongisde travis-ci ci(cirrus): use cirrus-ci alongisde travis-ci May 28, 2019
@myii myii force-pushed the ci/cirrus-alongside-travis branch from 00f61e7 to b05b0d2 Compare May 28, 2019 20:38
@myii myii force-pushed the ci/cirrus-alongside-travis branch from b05b0d2 to e3e75f5 Compare May 28, 2019 20:40
@javierbertoli javierbertoli merged commit 8c3ad3c into saltstack-formulas:master May 28, 2019
@myii myii deleted the ci/cirrus-alongside-travis branch May 28, 2019 21:13
@javierbertoli
Copy link
Member

Thanks for all this effort, @myii 🍺

@myii
Copy link
Member Author

myii commented May 28, 2019

Just a note for posterity, we enabled all 15 instances in the matrix for initial testing. We'll reduce back to the 5/6 after a short while.

Thanks for the merge @javierbertoli!

@aboe76
Copy link
Member

aboe76 commented May 28, 2019

This cirrus ci, looks good, clean interface @myii how do you like it so far?

@myii
Copy link
Member Author

myii commented May 28, 2019

@aboe76 I'm liking it, maybe for the wrong reasons so far! Once we get a few out there and we gain the concurrency benefits, I think I'm going to like it for the right reasons.

By the way, I've just added another Cirrus PR here: saltstack-formulas/syslog-ng-formula#15. Would you mind merging it when the tests complete?

@saltstack-formulas-travis

🎉 This PR is included in version 0.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants