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

Adding basic CircleCI configuration. #2701

Merged
merged 1 commit into from
Nov 9, 2016

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Nov 7, 2016

Trying out CircleCI in earnest since Travis throttles us every day.

Most of our tooling is Travis centric, so this alone is insufficient, but it's worth seeing if we can at least go green on Circle CI.

I have been working on https://github.com/dhermes/ci-diff-helper in the last few weeks, so hopefully I can factor out our Travis-dependent code and replace it with more generic "help me in any CI" code.

/cc @omaray @jgeewax @jonparrott

@dhermes dhermes added the build label Nov 7, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 7, 2016
@dhermes dhermes force-pushed the add-circleci-config branch 2 times, most recently from 2b9626b to aacd2e1 Compare November 7, 2016 23:26
@dhermes dhermes force-pushed the add-circleci-config branch from 2fccf32 to 8576e49 Compare November 8, 2016 00:38
@daspecster
Copy link
Contributor

Bump!

All our PR builds are red right now. Is there a way we could just run both to get this in and then maybe bring up dropping travis if we're happy with circleci?

@dhermes dhermes merged commit 43631a8 into googleapis:master Nov 9, 2016
@dhermes dhermes deleted the add-circleci-config branch November 9, 2016 16:22
@dhermes
Copy link
Contributor Author

dhermes commented Nov 9, 2016

This is failing to upload to coveralls.io for some reason. From the first successful merge build:

...
coveralls runtests: commands[18] | coveralls
Submitting coverage to coveralls.io...
Coverage submitted!
Couldn't find a repository matching this job.
'url'
ERROR: InvocationError: '/home/ubuntu/google-cloud-python/.tox/coveralls/bin/coveralls'
___________________________________ summary ____________________________________
ERROR:   coveralls: commands failed

scripts/coveralls.sh returned exit code 1

@dhermes
Copy link
Contributor Author

dhermes commented Nov 10, 2016

richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants