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

Add dependency fix for sudden BigQuery test failure #109

Merged
merged 1 commit into from
Oct 2, 2020

Conversation

vinceatbluelabs
Copy link
Contributor

No description provided.

@@ -105,6 +105,19 @@ def initialize_options(self) -> None:
#
# https://github.com/googleapis/google-auth-library-python/issues/190
'google-api-python-client>=1.8.0,<1.9.0',
#

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file, one of the 3 largest in the project, increased in size to 357 lines. The total size of those files is now 3399 lines (target: 1028). Is this file complex enough to refactor?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately we established you really can't break up stuff in setup.py without breaking some different packaging scenarios.

Copy link
Contributor

@cwegrzyn cwegrzyn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

#
# This seems to be a clue:
#
# https://github.com/pypa/pip/issues/8407
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Praying for the new resolution stuff to be out and stable soon-- although i'm sure it's going to break all sorts of back/forwards compatibility things, sigh.

@vinceatbluelabs vinceatbluelabs force-pushed the new_google-api-core-dependency branch from 0fb1493 to cb81f02 Compare October 2, 2020 14:26
@vinceatbluelabs vinceatbluelabs merged commit ba0c233 into master Oct 2, 2020
@vinceatbluelabs vinceatbluelabs deleted the new_google-api-core-dependency branch October 2, 2020 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants