Resolve UP043 TODO - #401
Avasam (Avasam) wants to merge 3 commits into
Conversation
|
🔒 Automated review in progress — Bill Schnurr (@bschnurr) is auto-reviewing this PR. |
|
Result: Verification detailsVerification: Isolated verification observed failures that were not classified as caused by this PR: Ruff checks for changed stub areas, Ruff formatting for changed stub areas, Dependency and test discovery. The relevant tests could not be fully run in the isolated environment; this review is not fully verified. Summary: The targeted UP043 Ruff check passed, and all 667 relevant stub files passed Ruff formatting. No tests were added by the PR. Full mypy, Pyright, and stubtest validation could not run because the sandbox lacked required dependencies and the repository provides no lockfile for permitted offline installation. Confidence is therefore partial; the only observed failures were environment-related or in an unchanged README snippet. Test runs: 2 passed, 3 failed, 3 not run
❌
|
|
Result: Verification detailsVerification: Isolated verification observed failures whose relationship to this PR could not be determined: Ruff repository lint, Mypy for affected stub packages; this review is not fully verified. Isolated verification observed failures that were not classified as caused by this PR: Ruff formatter. Summary: The focused UP043 lint check passed across all stubs. Full lint and formatting were blocked by unrelated baseline findings under Ruff 0.16.4 rather than the pinned 0.15 series. Targeted mypy, Pyright, and stubtest verification was blocked by missing offline dependencies and no lockfile. No tests were added, so verification is partial. Test runs: 2 passed, 3 failed, 2 not run
❌
|
Bill Schnurr (bschnurr)
left a comment
There was a problem hiding this comment.
Approved via Review Center.
Removes the 2nd and 3rd params of
Generatorif they areNoneorAny, matching typeshed (default typevar isNone, butAnys are auto insertion where it was unknown or didn't matter. typeshed simply collapses them to default).Also when 3rd param was
NoReturnwhilst overlapping an existing Union.