Skip to content

Conversation

@EepyElvyra
Copy link
Contributor

About

This PR corrects HTTPClient typehints and also adds a pyi

Checklist

@EepyElvyra EepyElvyra changed the title refactor: correct httpclient return typehints, change payload naming for consistency, refactor: correct HTTPClient return typehints, change payload naming for consistency, Mar 1, 2022
@EepyElvyra EepyElvyra changed the title refactor: correct HTTPClient return typehints, change payload naming for consistency, refactor: Make HTTP a folder and extract methods according to their endpoint Mar 1, 2022
@EepyElvyra EepyElvyra added breaking This contains breaking changes enhancement New feature or request labels Mar 1, 2022
@i0bs
Copy link
Contributor

i0bs commented Mar 1, 2022

this is the most unprofessional comment i can make but holy shit thank god we're finally doing this i've absolutely hated the original way we've been doing it thus far

@EepyElvyra
Copy link
Contributor Author

this is the most unprofessional comment i can make but holy shit thank god we're finally doing this i've absolutely hated the original way we've been doing it thus far

lets get this on starboard then

@EepyElvyra EepyElvyra requested a review from i0bs March 3, 2022 11:24
@EepyElvyra EepyElvyra enabled auto-merge (squash) March 5, 2022 21:38
@EepyElvyra EepyElvyra disabled auto-merge March 5, 2022 21:39
@EepyElvyra EepyElvyra requested a review from FayeDel March 5, 2022 21:39
@EepyElvyra EepyElvyra requested review from i0bs March 14, 2022 19:46
@EepyElvyra
Copy link
Contributor Author

All checks passed now

@i0bs i0bs merged commit d9cd5b6 into interactions-py:unstable Mar 17, 2022
@EepyElvyra EepyElvyra deleted the fix_http.py branch March 17, 2022 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking This contains breaking changes enhancement New feature or request pending Pending approve priority This Issue/PR must be resolved first before accepting any others

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

[REQUEST] Make HTTP a folder and extract methods according to their endpoint

2 participants