fix: remove dependency on types-requests

[rouge8]

Summary

Requests 2.34.0 and newer includes inline type hints.

Test plan

make typecheck

Issue number

Fixes #3507.

Checks

• I've added new tests (if relevant)
• I've added/updated the relevant documentation
• I've run make lint and make format
• I've made sure tests pass

[seratch]

Thanks for sharing this. requests 2.34 is pretty new, so we'd like to hold off removing types-requests at this moment.

[rouge8]

There shouldn't be any behavior changes for any users of the library. Keeping the dependency means that any users of the library can't upgrade requests and have correct type-checking behavior though.

[seratch]

Yes, you're right. We'll merge this change eventually, but I just wanted to wait and see the adoption for a short period of time.