Skip to content

Conversation

@parthea
Copy link
Contributor

@parthea parthea commented Mar 4, 2021

Fixes #1214 🦕

I was able to re-create the issue by using the following steps.

  1. Create a clean pyenv environment
  2. Download google-api-python-client-2.0.1 from PyPi here and unzip it
  3. Install it using pip install <Unzipped Directory>
  4. Confirm discovery.build() fails on any call

To Verify the fix, repeat the steps above, but apply the update in this PR prior to installing the library.

@parthea parthea requested a review from a team as a code owner March 4, 2021 16:30
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Mar 4, 2021
@parthea parthea requested a review from busunkim96 March 4, 2021 16:32
@busunkim96
Copy link
Contributor

Confirmed that the fix works for me locally. Thanks @parthea!

@parthea parthea added the automerge Merge the pull request once unit tests and other checks pass. label Mar 4, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit ad618d0 into master Mar 4, 2021
@gcf-merge-on-green gcf-merge-on-green bot deleted the fix-missing-documents-in-published-version branch March 4, 2021 17:02
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Mar 4, 2021
parthea added a commit that referenced this pull request Apr 20, 2021
parthea added a commit that referenced this pull request Apr 20, 2021
gcf-merge-on-green bot pushed a commit that referenced this pull request Apr 20, 2021
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/google-api-python-client/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [x] Ensure the tests and linter pass
- [x] Code coverage does not decrease (if any source code was changed)
- [x] Appropriate docs were updated (if necessary)

Fixes #1235 🦕

In an attempt to verify that this is a fix for #1235  I'm going push a commit that removes the fix from #1221 in this PR and confirm that the checks fail.
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.

Published package is missing discovery files in discovery_cache

2 participants