Skip to content
This repository has been archived by the owner on Nov 5, 2019. It is now read-only.

Sending the coverage report to coveralls in gae env. #560

Merged
merged 1 commit into from
Jul 20, 2016
Merged

Sending the coverage report to coveralls in gae env. #560

merged 1 commit into from
Jul 20, 2016

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Jul 20, 2016

This is because the GAE import failed in the py27 env.

@dhermes
Sending the coverage report to coveralls in gae env.
ffd6e09 
This is because the GAE import failed in the py27 env.
@dhermes
Copy link
Contributor Author

dhermes commented Jul 20, 2016

See #558

@dhermes dhermes mentioned this pull request Jul 20, 2016
Closed
@theacodes
Copy link
Contributor

This isn't the way I'd like to do this, I'd like to combine the reports from all runs. I'll merge this for now though.

@theacodes theacodes merged commit 3e9b857 into googleapis:master Jul 20, 2016
@dhermes
Copy link
Contributor Author

dhermes commented Jul 21, 2016

RE: combine from all runs, tox -e coveralls still does py27 and gae together, but not the Python 3 reports. Is that what you meant?

@dhermes dhermes deleted the restore-coveralls branch July 28, 2017 15:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@theacodes theacodes

Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dhermes @theacodes @googlebot @nathanielmanistaatgoogle