Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Appveyor: this-time-for-sure-rocky #1973

Merged
merged 1 commit into from
Jul 11, 2016
Merged

Appveyor: this-time-for-sure-rocky #1973

merged 1 commit into from
Jul 11, 2016

Conversation

tseaver
Copy link
Contributor

@tseaver tseaver commented Jul 11, 2016

Install GAX-related wrappers via pip to work around #1972.

Install GAX-related wrappers via pip to work around #1972.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 11, 2016
@tseaver
Copy link
Contributor Author

tseaver commented Jul 11, 2016

@daspecster I'm thinking to cancel the Travis build and just merge in order to poke Appveyor:

https://ci.appveyor.com/project/dhermes/gcloud-python/

@tseaver tseaver merged commit aa56c9f into googleapis:master Jul 11, 2016
@tseaver
Copy link
Contributor Author

tseaver commented Jul 11, 2016

Done. Now watching with bated breath:

https://ci.appveyor.com/project/dhermes/gcloud-python/build/1.0.487.master

@tseaver
Copy link
Contributor Author

tseaver commented Jul 11, 2016

@daspecster woohoo! We have a real test failure (rather than a build failure) on Appveyor:

======================================================================
FAIL: test_read_from_cli_info (gcloud.test__helpers.Test__get_default_service_project_id)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\projects\gcloud-python\gcloud\test__helpers.py", line 223, in test_read_from_cli_info
    self.assertEqual('test-project-id', project_id)
AssertionError: 'test-project-id' != None

@daspecster
Copy link
Contributor

yay! This looks familiar too!

@tseaver
Copy link
Contributor Author

tseaver commented Jul 11, 2016

@daspecster do we have an issue open for that?

@daspecster
Copy link
Contributor

Nope, but there is now!

This was referenced Aug 3, 2016
@tseaver tseaver deleted the 1863-appveyor-protobuf-this-time-for-sure branch September 8, 2016 01:11
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. grpc testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants