Skip to content
This repository has been archived by the owner on Jan 25, 2024. It is now read-only.

Drop Type Stubs that we no longer need #802

Merged
merged 4 commits into from
Aug 7, 2022
Merged

Conversation

trickeydan
Copy link
Contributor

Requesting @RealOrangeOne for review from Zoloto perspective

Checklist

  • I have updated the relevant documentation
  • I have added the semver-major, semver-minor or semver-patch label as appropriate
  • I would like multiple reviewers to approve my code before merging

Why do you want to make these changes?

We no longer require these stubs

Which parts of the codebase do your changes affect?

None, we are just using stubs that are not maintained by us

How do your changes affect student or volunteer experience?

No need to maintain stubs

Are your changes related to any existing issues or PRs? How so?

N/A

Zoloto has PEP561 types now.
Pytest is now typed
cached_property now has it's own types
We no longer use this library directly
@trickeydan trickeydan added the typing A incorrect or loose type that needs correcting. label Aug 5, 2022
@codeclimate
Copy link

codeclimate bot commented Aug 5, 2022

Code Climate has analyzed commit d9f7a64 and detected 0 issues on this pull request.

View more on Code Climate.

@trickeydan trickeydan merged commit 2d0066f into main Aug 7, 2022
@trickeydan trickeydan deleted the dgt/drop-unused-stubs branch August 7, 2022 14:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
typing A incorrect or loose type that needs correcting.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants