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

SUPP: update min google-cloud-bigquery version #2304

Merged
merged 1 commit into from
Aug 13, 2020

Conversation

tswast
Copy link
Collaborator

@tswast tswast commented Jul 31, 2020

This is needed in order to set the application name in the user agent
header. This feature to be added in:
#2303

Also, removes pin on google-api-core, now that
googleapis/python-api-core#47 is closed.

Copy link
Contributor

@datapythonista datapythonista left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @tswast

docs/source/release/index.rst Outdated Show resolved Hide resolved
@tswast
Copy link
Collaborator Author

tswast commented Aug 4, 2020

Windows test failure seems unrelated to this change

Chocolatey installed 0/1 packages. 1 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Failures
 - mariadb - mariadb not installed. The package was not found with the source(s) listed.
 Source(s): 'https://chocolatey.org/api/v2/'

@datapythonista
Copy link
Contributor

Yes, thanks. I'm aware, the whole CI is broken again. I'm trying to simplify things in the CI, so it breaks less, or at least is easier to fix when it breaks, but it'll take me some time.

@datapythonista
Copy link
Contributor

@tswast you'll have to redo this, sorry. All those requirement files do not exist anymore, now bigquery dependencies are in ci/deps/bigquery.yml.

This is needed in order to set the application name in the user agent
header. This feature to be added in:
ibis-project#2303

Also:

* Remove pin on google-api-core, now that
  googleapis/python-api-core#47 is closed.
* Update docs/source/release/index.rst

Co-authored-by: Marc Garcia <garcia.marc@gmail.com>
@tswast
Copy link
Collaborator Author

tswast commented Aug 13, 2020

All those requirement files do not exist anymore, now bigquery dependencies are in ci/deps/bigquery.yml.

Done!

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

lgtm. can you add a note in the release notes

@jreback jreback added this to the Next Feature Release milestone Aug 13, 2020
@datapythonista
Copy link
Contributor

lgtm. can you add a note in the release notes

It's there, I think you checked the diff of the last commit ;)

@jreback
Copy link
Contributor

jreback commented Aug 13, 2020

great, lgtm.

@datapythonista datapythonista merged commit 1a7a842 into ibis-project:master Aug 13, 2020
@datapythonista
Copy link
Contributor

Thanks @tswast

LeeTZ pushed a commit to LeeTZ/ibis that referenced this pull request Sep 4, 2020
This is needed in order to set the application name in the user agent
header. This feature to be added in:
ibis-project#2303

Also:

* Remove pin on google-api-core, now that
  googleapis/python-api-core#47 is closed.
* Update docs/source/release/index.rst

Co-authored-by: Marc Garcia <garcia.marc@gmail.com>
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