opencensus-python icon indicating copy to clipboard operation
opencensus-python copied to clipboard

Travis Integration

Open bu3alwa opened this issue 5 years ago • 3 comments

Continuing off #402, I tried to follow in opentelemetary tox.ini. Right now the only two failing tests in with tox are lint and docs and both fail due to warnings.

With the added eachdist.py from opentelemetry we get a warning when black . --diff --check is run. There would be reformating needed for the warnings to go away and the tests to succeed.

The second warning is when building the docs its missing opencensus.ext.

Since I am new to this project I am not sure how to proceed. For the first warning is it safe to reformat the source with black? And then for the second I am not sure why ext is missing from opencensus module. To run those specific test tox -e lint and tox -e docs

bu3alwa avatar Mar 07 '20 17:03 bu3alwa

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

googlebot avatar Mar 07 '20 17:03 googlebot

@googlebot I signed it!

bu3alwa avatar Mar 07 '20 17:03 bu3alwa

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

googlebot avatar Mar 07 '20 17:03 googlebot