-
Notifications
You must be signed in to change notification settings - Fork 148
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
Travis -> CircleCI #133
Comments
Would love to have a crack at this, but it looks like I need repo admin to get started. @SuperQ would you be able to help with that when you have a minute? |
@SuperQ Just running through old issues and I'd like to give this one a go. Figure it makes sense for the reasons outlined by @lawrencejones and the fact that prometheus/prometheus uses CircleCI too. Any chance I could (at least temporarily) be made admin on this repo? |
Looking at the other repos, it looks like they also use Circle, so this'd make sense to me |
I wonder, however, if it'd make more sense to use the "Matrix" feature, like the Python repo does, instead of specifying the different versions like in Lawrence's proposal. Although that's a nitpick, and I may be over-reading on what he was proposing to do |
I don't think Lawrence was being that specific about what Circle config we write as much as saying that we should move over to Circle. Not least because they added build matrices almost a year after he raised this issue. |
Closing this as it's been done in #230. |
Travis has gone through some unfortunate times recently and CircleCI offer a more comprehensive and flexible configuration in their V2 spec.
It could be good to run CI like this: https://github.com/gocardless/statesman/blob/master/.circleci/config.yml
The text was updated successfully, but these errors were encountered: