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

chore: sync v3 with main branch - #1055

Merged
tswast merged 4 commits into
googleapis:v3from
plamut:sync-v3-with-main
Nov 10, 2021
Merged

chore: sync v3 with main branch#1055
tswast merged 4 commits into
googleapis:v3from
plamut:sync-v3-with-main

Conversation

@plamut

Copy link
Copy Markdown
Contributor

Another sync merge after merging type annotations into the main branch. This is to reach as clean a state as possible before adding new features and bugfixes to the v3 branch.

PR checklist:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

plamutand others added 4 commits November 8, 2021 10:21
* process: add mypy types check to nox sessions
* Ignore type errors for not annotated modules
Several dependencies lack type annotations, or they don't advertise
themselves as type-annotated. We do not want `mypy` to complain about
these.
* Fix mypy complaints (batch 1)
* Fix mypy complaints (batch 2)
* Fix mypy complaints (batch 3)
* Fix mypy false positive errors
* Simplify external config options instantiation
* Do not ignore api-core in type checks
More recent releases of google-api-core have typing enabled.
* Remove unneeded __hash__ = None lines
* Use an alias for timeout type in client.py
* Fix PathLike subscription error in pre-Python 3.9
* Fix a typo in docstring
Co-authored-by: Tim Swast <swast@google.com>
* Add mypy to the list of nox sessions to run
* Fix opentelemetry type error
The Opentelemetry APi has changed from the minimum version the
BigQuery client currently uses, we thus need to bound the maximum
Opentelemetry version.
In addition, that maximum version does not yet support type checks, thus
it is ignored.
* 🦉 Updates from OwlBot
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Exclude type-checking code from coverage
* Fix patching opentelemetry tracer pvoider
* Adjust get_job() return type, ignore opentelemetry
Co-authored-by: Tim Swast <swast@google.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
@plamut
plamut requested review from a team and tswastNovember 9, 2021 20:09
@product-auto-labelproduct-auto-labelBot added the api: bigquery Issues related to the googleapis/python-bigquery API. label Nov 9, 2021
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Nov 9, 2021
@plamut

Copy link
Copy Markdown
ContributorAuthor

The do not merge label is a reminder to use a merge commit when merging (without squashing!).

@plamutplamut added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 9, 2021
@tswasttswast removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 10, 2021
@tswast
tswast merged commit 42d3db6 into googleapis:v3Nov 10, 2021
@plamut
plamut deleted the sync-v3-with-main branch November 10, 2021 20:20
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigqueryIssues related to the googleapis/python-bigquery API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@plamut@tswast