coveragepy icon indicating copy to clipboard operation
coveragepy copied to clipboard

WIP type annotations

Open KotlinIsland opened this issue 3 years ago • 3 comments

Very WIP

Contains:

  • partial type annotations for some modules
  • github workflow integration

addresses: #1434

KotlinIsland avatar Aug 29 '22 11:08 KotlinIsland

I haven't looked at this closely, but do you have a way to run mypy as part of the tools?

nedbat avatar Aug 29 '22 12:08 nedbat

I haven't looked at this closely, but do you have a way to run mypy as part of the tools?

I'm unfamiliar with your project configuration (make, pip-compile) but I can give it a go.

KotlinIsland avatar Aug 29 '22 13:08 KotlinIsland

Thanks for taking this all on. I added some details to the Contributing section of the docs about dependencies and make upgrade, though I wonder if I've added enough? Can you take a look and let me know? Thanks so much.

nedbat avatar Aug 30 '22 00:08 nedbat

What is the future of this pull request? It might be too hard to get everything type-checked, since the master branch is continuing development. Is there a way to get the tooling in place, and gradually make progress on adding the annotations?

nedbat avatar Nov 06 '22 13:11 nedbat

Hmm, sure. I could clean this up and get a minimal state ready to merge when I get time.

KotlinIsland avatar Nov 06 '22 14:11 KotlinIsland

What is the future of this pull request? It might be too hard to get everything type-checked, since the master branch is continuing development. Is there a way to get the tooling in place, and gradually make progress on adding the annotations?

@nedbat On uvicorn, we created an issue to track our progress: https://github.com/encode/uvicorn/issues/998.

Each PR had an entry on the mypy configuration file (setup.cfg for us).

I can create a similar issue for coverage, and help to coordinate, if wanted.

Kludex avatar Nov 09 '22 18:11 Kludex

I started a more gradual approach. As of 212f6520, four files on master are checked with mypy. I'd be happy to have help adding more files gradually, and eventually making the checks more strict and complete. I'll close this PR now, but feel free to re-open it if you can make this way work with the latest code.

nedbat avatar Dec 27 '22 22:12 nedbat

As of now (commit b059a67f), all of the files in coverage/ and tests/ are checked by mypy.

nedbat avatar Jan 13 '23 03:01 nedbat