Skip to content

chore: type check with mypy --check-untyped-defs option - #693

Merged
tswast merged 18 commits into
mainfrom
mypy-check-untyped-defs
May 15, 2024
Merged

chore: type check with mypy --check-untyped-defs option#693
tswast merged 18 commits into
mainfrom
mypy-check-untyped-defs

Conversation

@tswast

@tswast tswast commented May 15, 2024

Copy link
Copy Markdown
Collaborator

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:

  • 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)

Fixes #<issue_number_goes_here> 🦕

@product-auto-label product-auto-label Bot added size: xs Pull request size is extra small. api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. labels May 15, 2024
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. and removed size: xs Pull request size is extra small. labels May 15, 2024
@tswast
tswast marked this pull request as ready for review May 15, 2024 18:09
@tswast
tswast requested review from a team and chelsea-lin May 15, 2024 18:09
@product-auto-label product-auto-label Bot added size: l Pull request size is large. and removed size: m Pull request size is medium. labels May 15, 2024

@chelsea-lin chelsea-lin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@property
def result(self):
return self.axes
# TODO(b/): fix type error

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: b/340896123

@tswast
tswast enabled auto-merge (squash) May 15, 2024 21:08
@tswast
tswast merged commit 9fee857 into main May 15, 2024
@tswast
tswast deleted the mypy-check-untyped-defs branch May 15, 2024 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants