Switch to codecov.io by daspecster · Pull Request #1876 · googleapis/google-cloud-python · GitHub

No description provided.

Possibly an example when it finishes generating...
https://codecov.io/github/daspecster/gcloud-python/commit/8f90abc07f33c0b84522ae58377cdcbaca569f65

Current coverage is 100%

No coverage report found for master at bd6435a.

Powered by Codecov. Last updated by bd6435a...6d08b77

Yeah still playing with it.
I need to get the coverage xml generated before codecov runs.

On Monday, June 20, 2016, Codecov notifications@github.com wrote:

Current coverage
https://codecov.io/gh/GoogleCloudPlatform/gcloud-python/pull/1876?src=pr
is 100%

No coverage report found for master at 986302f
986302f
.

Powered by Codecov https://codecov.io?src=pr. Last updated by
986302f...60c0364
https://codecov.io/gh/GoogleCloudPlatform/gcloud-python/compare/986302f3db5ab71e0951fa4fab796333d7d7cece...60c03644ff146537e351822b77bd4a27732c06aa

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1876 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AALvyGawrhgLR23ki28X9RDdsOmdeqt7ks5qNwP7gaJpZM4I6FhG
.

Tom Schultz

@dhermes the last build worked.

Travis:
https://travis-ci.org/GoogleCloudPlatform/gcloud-python/builds/139037369#L778

codecov:
https://codecov.io/gh/GoogleCloudPlatform/gcloud-python/tree/60c03644ff146537e351822b77bd4a27732c06aa/gcloud

LGTM, please squash. Also we should make sure codecov.io adds a check to this PR before merging

Also the coveralls badge should be replaced

@dhermes, I don't have admin rights to the repo to add a github check.

Looks like we're not alone!
https://codecov.io/gh/googlecloudplatform

@daspecster Which page do you need admin rights for? I'm on https://github.com/GoogleCloudPlatform/gcloud-python/settings/hooks trying to add the service but can't find it

OK figured it out on https://codecov.io/gh/GoogleCloudPlatform/gcloud-python/settings

Here is the badge FYI

.. image:: https://codecov.io/gh/GoogleCloudPlatform/gcloud-python/branch/master/graph/badge.svg
  :target: https://codecov.io/gh/GoogleCloudPlatform/gcloud-python

@daspecster Now that the hook has been added, can you add a commit with the badge so we can test it?

Awesome! I was looking.

I put http://codecov.io/github/googlecloudplatform/gcloud-python/coverage.svg?branch=master for the badge target in the readme but I'll change it to yours.

I just copied that RST snippet from https://codecov.io/gh/GoogleCloudPlatform/gcloud-python/settings/badge Where'd you get the other one?

I think I saw it in their docs somewhere.

codecov runtests: commands[1] | codecov
      _____          _
     / ____|        | |
    | |     ___   __| | ___  ___ _____   __
    | |    / _ \ / _  |/ _ \/ __/ _ \ \ / /
    | |___| (_) | (_| |  __/ (_| (_) \ V /
     \_____\___/ \____|\___|\___\___/ \_/
                                    v2.0.5
==> Detecting CI provider
  -> Got branch from git/hg
  -> Got sha from git/hg
==> Preparing upload
Error: Missing repository upload token
Tip: See all example repositories: https://github.com/codecov?query=example
Support channels:
  Email:   hello@codecov.io
  IRC:     #codecov
  Gitter:  https://gitter.im/codecov/support
  Twitter: @codecov

Still no check, though it worked

codecov runtests: commands[1] | codecov
      _____          _
     / ____|        | |
    | |     ___   __| | ___  ___ _____   __
    | |    / _ \ / _  |/ _ \/ __/ _ \ \ / /
    | |___| (_) | (_| |  __/ (_| (_) \ V /
     \_____\___/ \____|\___|\___\___/ \_/
                                    v2.0.5
==> Detecting CI provider
    Travis Detected
==> Preparing upload
==> Processing gcov (disable by -X gcov)
    Executing gcov (find /home/travis/build/GoogleCloudPlatform/gcloud-python -type f -name '*.gcno'  -exec gcov -pb  {} +)
==> Collecting reports
    Generating coverage xml reports for Python
    + /home/travis/build/GoogleCloudPlatform/gcloud-python/coverage.xml bytes=1274444
==> Appending environment variables
    + TRAVIS_OS_NAME
==> Uploading
    .url https://codecov.io
    .query commit=51af5e55de35c61123d61d8cc1eb76eca06b3333&service=travis&pr=1876&package=py2.0.5&job=139057017&branch=master&slug=GoogleCloudPlatform%2Fgcloud-python&build=3845.1
    Pinging Codecov...
    Uploading to S3...

Hmmm

@dhermes after some digging, I think we need that codecov.yml file there to get it to trigger status checks. Let me know if one shows up now?

@dhermes, can you add the integration here https://github.com/integrations/codecov to the GoogleCloudApplication org if possible.
There might be something to do with the third-party restrictions as well.

I think https://github.com/organizations/GoogleCloudPlatform/settings/oauth_application_policy
and/or
https://github.com/organizations/GoogleCloudPlatform/settings/applications/

CodeCov uses all github permissions so I think we have to get it setup on the org side.

I don't know if I have admin rights / don't see any "Add" button on the integrations page (I visited it multiple times yesterday trying to figure out what to click).

Would this be something @jgeewax might have to do?

Maybe

@daspecster Where did the previous post from the codecov bot come from?

If you change this value it will post a comment.
https://github.com/GoogleCloudPlatform/gcloud-python/pull/1876/files#diff-8bc9db6adba7208e25c96be322351063R1

That link anchors weirdly...it's line 1 of codecov.yml.

@jgeewax, if you're game, can we enable codecov in the organization apps/oauth?

👍

Awesome thanks!

Hmmm, still not showing up in here. Could someone check and see if codecov is in the projected branch settings now?

I tried to reset the webhook.
Looks like there is still an issue on the github permissions side.

FWIW, the webhook is already in place.

@tseaver, if you go here can you click on the "reset webhook" button with any success?

If that doesn't start working then we could also just turn the integration back on to get codecov to post a comment about the coverage.

I'm thinking that this might have to get into master to fully work. We could change this so it does both coveralls and codecov to test it.

Can you click on the "reset webhook" button with any success?

Nope, I get the same error you posted.

If that doesn't start working then we could also just turn the integration back on to get codecov to post a comment about the coverage.

I don't like those comments: I would rather have it be a "check".

I'm thinking that this might have to get into master to fully work. We could change this so it does both coveralls and codecov to test it.

Let's try that. We can drop coveralls in a follow-up PR.

LGTM

Change codecov settings.

I squashed some of the commits and when travis passes I'll merge.

…o_dataframe` (#1876) * add regression tests for empty dataframe * fix arrow test to be compatible with old pyarrow

Co-authored-by: Tim Sweña (Swast) <swast@google.com>

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 20, 2016

daspecster mentioned this pull request Jun 20, 2016

Have travis-ci run tox -e cover to catch code branch coverage misses #1872

Closed

dhermes reviewed Jun 20, 2016
View reviewed changes

Comment thread .travis.yml Outdated

after_success:

- tox -e coveralls

- bash <(curl -s https://codecov.io/bash)

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

daspecster force-pushed the switch-to-codecovio branch 2 times, most recently from 5915215 to 60c0364 Compare June 20, 2016 21:14

dhermes reviewed Jun 20, 2016
View reviewed changes

Comment thread tox.ini Outdated

basepython =

python2.7

passenv =

CI TRAVIS_BUILD_ID TRAVIS TRAVIS_BRANCH TRAVIS_JOB_NUMBER TRAVIS_PULL_REQUEST TRAVIS_JOB_ID TRAVIS_REPO_SLUG TRAVIS_COMMIT

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

Switch to codecov.io closes googleapis#1872

1d5973e

daspecster force-pushed the switch-to-codecovio branch from b2ff4f2 to 1d5973e Compare June 20, 2016 22:12

tseaver added the testing label Jul 11, 2016

daspecster force-pushed the switch-to-codecovio branch from d9999f4 to 6d08b77 Compare July 14, 2016 15:54

daspecster added 3 commits July 14, 2016 14:02

Update badge url.

8967379

Add codecov.yml to trigger status check.

d6db37b

Change codecov settings.

Add coveralls back while testing codecov.

58cdf26

daspecster force-pushed the switch-to-codecovio branch from 6d08b77 to 58cdf26 Compare July 14, 2016 18:02

daspecster merged commit 1942f2d into googleapis:master Jul 14, 2016

daspecster mentioned this pull request Jul 18, 2016

Code branch test coverage #1993

Closed

This was referenced Aug 3, 2016

Prepare 0.18.0 release #2044

Closed

Prepare 0.18.0 release. #2048

Merged

daspecster deleted the switch-to-codecovio branch January 24, 2017 15:18

parthea pushed a commit that referenced this pull request Nov 26, 2025

chore(tests): update secret (#1876 )

01c25c1

parthea pushed a commit that referenced this pull request Mar 6, 2026

fix: avoid "Unable to determine type" warning with JSON columns in `t…

b3f7595

…o_dataframe` (#1876) * add regression tests for empty dataframe * fix arrow test to be compatible with old pyarrow

parthea pushed a commit that referenced this pull request Apr 1, 2026

docs: update gsutil commands to gcloud commands (#1876 )

32d6a40

Co-authored-by: Tim Sweña (Swast) <swast@google.com>

               after_success:
-                - tox -e coveralls
+                - bash <(curl -s https://codecov.io/bash)

+              basepython =
+                  python2.7
+              passenv =
+                  CI TRAVIS_BUILD_ID TRAVIS TRAVIS_BRANCH TRAVIS_JOB_NUMBER TRAVIS_PULL_REQUEST TRAVIS_JOB_ID TRAVIS_REPO_SLUG TRAVIS_COMMIT

Switch to codecov.io by daspecster · Pull Request #1876 · googleapis/google-cloud-python · GitHub

Uh oh!

Switch to codecov.io - #1876

Switch to codecov.io#1876daspecster merged 4 commits intogoogleapis:mastergoogleapis/google-cloud-python:masterfrom daspecster:switch-to-codecoviodaspecster/google-cloud-python:switch-to-codecovioCopy head branch name to clipboard

Conversation

daspecster commented Jun 20, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

daspecster commented Jun 20, 2016

Uh oh!

codecov-io commented Jun 20, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Current coverage is 100%

Uh oh!

daspecster commented Jun 20, 2016

Uh oh!

daspecster commented Jun 20, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

dhermes commented Jun 20, 2016

Uh oh!

dhermes commented Jun 20, 2016

Uh oh!

daspecster commented Jun 20, 2016

Uh oh!

daspecster commented Jun 20, 2016

Uh oh!

dhermes commented Jun 20, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

dhermes commented Jun 20, 2016

Uh oh!

dhermes commented Jun 20, 2016

Uh oh!

daspecster commented Jun 20, 2016

Uh oh!

dhermes commented Jun 20, 2016

Uh oh!

daspecster commented Jun 20, 2016

Uh oh!

dhermes commented Jun 20, 2016

Uh oh!

dhermes commented Jun 20, 2016

Uh oh!

daspecster commented Jun 20, 2016

Uh oh!

daspecster commented Jun 21, 2016

Uh oh!

daspecster commented Jun 21, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

dhermes commented Jun 21, 2016

Uh oh!

daspecster commented Jun 21, 2016

Uh oh!

dhermes commented Jun 21, 2016

Uh oh!

dhermes commented Jun 21, 2016

Uh oh!

daspecster commented Jun 21, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

daspecster commented Jun 22, 2016

Uh oh!

jgeewax commented Jul 5, 2016

Uh oh!

daspecster commented Jul 5, 2016

Uh oh!

daspecster commented Jul 6, 2016

Uh oh!

daspecster commented Jul 13, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Switch to codecov.io#1876
daspecster merged 4 commits into
googleapis:masterfrom
daspecster:switch-to-codecovio

codecov-io commented Jun 20, 2016 •
edited
Loading

dhermes commented Jun 20, 2016 •
edited
Loading

daspecster commented Jun 21, 2016 •
edited
Loading

daspecster commented Jun 21, 2016 •
edited
Loading

daspecster commented Jul 13, 2016 •
edited
Loading