Skip to content
This repository was archived by the owner on Mar 6, 2026. It is now read-only.

chore: synchronize with Python templates#6

Merged
tswast merged 25 commits intomainfrom
tswast-templates
Apr 11, 2024
Merged

chore: synchronize with Python templates#6
tswast merged 25 commits intomainfrom
tswast-templates

Conversation

@tswast
Copy link
Copy Markdown
Contributor

@tswast tswast commented Apr 9, 2024

Please review #8 first! This PR depends on changes made in that PR.

Only 1 file here is hand-written:

  • owlbot.py

@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/python-bigquery-magics API. label Apr 9, 2024
@tswast
Copy link
Copy Markdown
Contributor Author

tswast commented Apr 9, 2024

Might be a good idea to get setup.py and unit tests working before continuing with this PR.

@tswast tswast added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 11, 2024
@tswast
Copy link
Copy Markdown
Contributor Author

tswast commented Apr 11, 2024

Marking as do not merge to wait for #8 to merge first.

@tswast tswast removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 11, 2024
@tswast tswast added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 11, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 11, 2024

- The feature must work fully on the following CPython versions:
3.7, 3.8, 3.9, 3.10, 3.11 and 3.12 on both UNIX and Windows.
3.7, 3.8, 3.11 and 3.12 on both UNIX and Windows.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Curious: why the de-emphasis on versions 3.9 and 3.10?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think this is taking the unit test versions from owlbot. We do want it to work on these, but I wonder if it's worth testing against so many versions?

@chalmerlowe
Copy link
Copy Markdown

Please review #8 first! This PR depends on changes made in that PR.

Only 1 file here is hand-written:

  • owlbot.py

I don't see owlbot.py in this PR.

Copy link
Copy Markdown
Contributor

@chelsea-lin chelsea-lin left a comment

Choose a reason for hiding this comment

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

LGTM overall

@tswast
Copy link
Copy Markdown
Contributor Author

tswast commented Apr 11, 2024

I don't see owlbot.py in this PR.

Oops. Merged that in #8

The owlbot lock files actually activates it.

@tswast tswast merged commit 373c072 into main Apr 11, 2024
@tswast tswast deleted the tswast-templates branch April 11, 2024 22:16
ashleyxuu pushed a commit that referenced this pull request Apr 12, 2024
* chore: synchronize with Python templates

* add owlbot configuration

* remove unknown paths from linter

* use templated noxfile

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* multiple teams: see: googleapis/synthtool#946

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* chore: grant api-bigquery and api-bigquery-dataframe teams write access to repo

* add python 3.7

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* chore: copy magics unit tests from google-cloud-bigquery

* include google-cloud-testutils in unittests

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* use bqstorage extras

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* use extras by python for everything

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* update tests to use bigquery_magics module

* remove auth from unit tests

* fix coverage

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigquery Issues related to the googleapis/python-bigquery-magics API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants