Skip to content

Conversation

@daspecster
Copy link
Contributor

@daspecster daspecster commented Jul 12, 2016

@tseaver not sure how I feel about the NO COVER's. The only other option I could think of was skipping tests on windows.

Appveyor: https://ci.appveyor.com/project/daspecster/gcloud-python-utjkc

LMKWYT

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 12, 2016
@daspecster
Copy link
Contributor Author

daspecster commented Jul 12, 2016

gahhh...lint ungrouped-imports? I set it to ignore those?

@daspecster daspecster force-pushed the fix-default-project-id-on-windows branch 2 times, most recently from 709f2a5 to 100ac64 Compare July 12, 2016 05:04
@tseaver
Copy link
Contributor

tseaver commented Jul 12, 2016

@daspecster I just merged #1974 to master, which should resolve the pylint problems for you.

@daspecster daspecster force-pushed the fix-default-project-id-on-windows branch from 100ac64 to bbfefee Compare July 12, 2016 14:07
@tseaver
Copy link
Contributor

tseaver commented Jul 12, 2016

LGTM

@daspecster daspecster merged commit de6252e into googleapis:master Jul 12, 2016
@daspecster daspecster deleted the fix-default-project-id-on-windows branch July 12, 2016 17:18
@tseaver
Copy link
Contributor

tseaver commented Jul 12, 2016

Ugh, still blows up trying to install grpcio on Python 3.4/3.5. Looks like there are no Py3k-compatible wheels on PyPI, which means it falls back to building from source, and hence needs protoc.

@daspecster
Copy link
Contributor Author

yeah I noticed that...
Doesn't look like they have the py3 stuff yet.
grpc/grpc#7135

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.

3 participants