-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[MAINTENANCE] Pin mypy
to 0.990
#6361
Conversation
✅ Deploy Preview for niobium-lead-7998 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
👇 Click on the image for a new way to code review
Legend |
@@ -752,7 +752,6 @@ def show_expectations_by_expectation_type( | |||
pprint.pprint( | |||
object=pprint_objects, | |||
indent=2, | |||
sort_dicts=False, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NathanFarmer
This wasn't added until python 3.8 😢
https://docs.python.org/3/library/pprint.html#pprint.PrettyPrinter
@@ -13,6 +13,7 @@ skip_gitignore = true | |||
extend_skip_glob = ['venv/*', 'docs/*'] | |||
|
|||
[tool.mypy] | |||
python_version = "3.7" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should help a bit with getting consistent errors across dev environments.
If you are using a python 3.10
feature that doesn't exist in 3.7
mypy should warn you.
* develop: [MAINTENANCE] Update column_reflection_fallback to also use schema name for Trino (#6350) [MAINTENANCE] Misc cleanup of GX Cloud helpers (#6352) [MAINTENANCE] Pin `mypy` to `0.990` (#6361) Subject: Support to include ID/PK in validation result for each row t… (#5876) [MAINTENANCE] Fix computed metrics type hint in ExecutionEngine.resolve_metrics() method (#6347) [MAINTENANCE] Refactor `usage_stats_opt_out` method in DataContext (#5339) Zep PostgresDatasource returns a list of batches. (#6341) [DOCS] Add `yarn snippet-check` command (#6351) [MAINTENANCE] Refactor out `termcolor` dependency (#6348) [MAINTENANCE] Move Cloud-specific enums to `cloud_constants.py` (#6349)
Changes proposed in this pull request:
The following changes should make our typing CI step much more consistent and reliable.
Pin mypy version to prevent CI breakage - as recommended by
mypy
Update
mypy
config to always run as if it was python 3.7 (our oldest supported version)pprint
related bug - https://docs.python.org/3/library/pprint.html#pprint.PrettyPrinter