search

bloomberg / pytest-memray

pytest plugin for easy integration of memray memory profiler
https://pytest-memray.readthedocs.io/en/latest/
Apache License 2.0
353 stars 25 forks source link

Drop `# type: ignore` comments for `@hookimpl` #95

Closed godlygeek closed 1 year ago

godlygeek commented 1 year ago

These no longer trigger a warning with the current versions of mypy, pytest, and pluggy. It looks as though these calls have been typed since pluggy 1.2.0, which was released 3 months ago.

We didn't notice this because our lint checks weren't scheduled to run on a schedule. This PR fixes that as well.

godlygeek commented 1 year ago

Mind taking another look at this @gusmonod ?

I needed to push another commit to address some test flakiness on main, and that dismissed your approval.