| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Bumps [mypy](https://github.com/python/mypy) from 1.4.1 to 1.18.1. - [Changelog](https://github.com/python/mypy/blob/master/CHANGELOG.md) - [Commits](python/mypy@v1.4.1...v1.18.1) --- updated-dependencies: - dependency-name: mypy dependency-version: 1.18.1 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
|
|
||
| def get_annotations(class_dict: Dict[str, Any]) -> Dict[str, Any]: | ||
| return class_dict.get("__annotations__", {}) | ||
| return class_dict.get("__annotations__", {}) # type: ignore[no-any-return] |
There was a problem hiding this comment.
As on L425, where we're already ignoring it, the error is
Returning Any from function declared to return "dict[str, Any]" [no-any-return]
Sorry, something went wrong.
|
Huh. On Python 3.9 the CI is failing with
But I don't see a single X | Y in main.py 🤔 |
Sorry, something went wrong.
|
The mypy check also takes super long, both on CI as locally. Weird - I keep investigating. |
Sorry, something went wrong.
There was a problem hiding this comment.
The false positive notifications of the X | Y syntax go away when specifying from __future__ import annotations at the top of the file, as suggested here.
But even if this PR goes green, we shouldn't merge it. The mypy runs are still taking a ridiculous amount of time, something like 12-13 minutes. Each full CI run usually takes less than 1 minute when looking at current open PRs.
Sorry, something went wrong.
|
With mypy 1.10.0 the execution time is still reasonable (but it's still a pretty old version of mypy - I'll keep investigating) Update: ignoring sqlmodel.sql._expression_select_gen.py from the mypy checks avoids the minute-long run. |
Sorry, something went wrong.
|
📝 Docs preview for commit f30beb2 at: https://d085961e.sqlmodel.pages.dev |
Sorry, something went wrong.
📝 Docs previewLast commit d787c3f at: https://76cacab2.sqlmodel.pages.dev |
Sorry, something went wrong.
|
🎉 |
Sorry, something went wrong.
|
from __future__ import annotations added by this PR causes Pyright in strict mode to show Type of "Field" is partially unknown error: #1598 |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Bumps mypy from 1.4.1 to 1.18.1.
ChangelogSourced from mypy's changelog.
... (truncated)
CommitsDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR: