Remove vendor proto files and use opencensus-proto instead. #596
Conversation
There was a problem hiding this comment.
LGTM. Now we can remove the code coverage exemption at https://github.com/census-instrumentation/opencensus-python/blob/master/.coveragerc#L8.
| @@ -41,6 +41,7 @@ | |||
| install_requires=[ | |||
| 'grpcio >= 1.0.0, < 2.0.0', | |||
| 'opencensus >= 0.4.dev0, < 1.0.0', | |||
| 'opencensus-proto >= 0.0.dev0, < 1.0.0', | |||
There was a problem hiding this comment.
FYI we've been replacing "dev" in the release branches. If opencensus-proto follows the same versioning strategy the first release will be 0.1.0 and the version number on master after the release should be 0.2.dev0, in which case this should be 0.1.dev0.
|
@songy23 just realized there was a lint rule which we can get rid off. You can leave it to me since we'll move from flake8 to pylint, up to you. https://github.com/census-instrumentation/opencensus-python/blob/master/nox.py#L119 |
|
Got it, I'll leave it to you then because I'm putting this PR on hold until we made the next release for opencensus-proto (currently it's 0.0.dev0). |
songy23
force-pushed
the
remove-ocagent-gen-files
branch
from
b026c64
to
58dfe4d
Compare
songy23
changed the title
|
census-instrumentation/opencensus-proto#197 is merged, now I think we can move forward with this PR. |
DO NOT MERGE until census-instrumentation/opencensus-proto#197 is merged and a new release is cut.Fixes #318.