setuptools icon indicating copy to clipboard operation
setuptools copied to clipboard

Initial pyright config

Open Avasam opened this issue 1 year ago • 2 comments

Summary of changes

Work towards step 2 of https://github.com/pypa/setuptools/issues/2345#issuecomment-1627561833 . Non-trivial fixes should go in a different PR. Merge https://github.com/pypa/setuptools/pull/3979 and #4257 first. npx -y pyright@latest . --pythonversion=3.8|3.11 --pythonplatform=Linux|Windows results: 0 errors, 418 warnings, 0 informations

Pull Request Checklist

Avasam avatar Jan 25 '24 02:01 Avasam

I understand that this is a [wip] but I left a few comments, hopefully that is fine with you.

Perfectly fine! I'm just waiting for a merge on #3979 (there's some overlap) before publishing the PR. I especially need guidance on running type-checkers (mypy/pyright) on the CI (tox/pytest setup) to prevent regressions.

Avasam avatar Jan 25 '24 21:01 Avasam

The hundreds (763) of information: Analysis of function "..." is skipped because it is unannotated are probably quite undesirable (we just started an error to remove spam in logs from EncodingWarning...) so I've opened https://github.com/microsoft/pyright/issues/7415

Edit: Looks like there's a lot less errors than when I initially tried this, so "analyzeUnannotatedFunctions": false wasn't really necessary.

Avasam avatar Mar 06 '24 21:03 Avasam

Thank you very much, let's get this one going.

abravalheri avatar Aug 27 '24 10:08 abravalheri