Skip to content

Stubs for slumber #5764

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Jul 12, 2021
Merged

Stubs for slumber #5764

merged 7 commits into from
Jul 12, 2021

Conversation

exemplary-citizen
Copy link
Contributor

A fairly popular object-oriented interface to ReSTful APIs. It acts as a wrapper around the requests library and abstracts away the handling of URLs, serialization, and request processing.

I started with stubgen, then cleaned up to address missing functions. Testing plan is to keep an eye on CI and fix any issues that arise.

cc: @JukkaL

@srittau
Copy link
Collaborator

srittau commented Jul 12, 2021

This needs to be added to pyrightconfig.stricter.json.

Copy link
Collaborator

@srittau srittau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good. One comment about imports below.

@@ -0,0 +1,13 @@
from .utils import iterator as iterator, url_join as url_join
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stubgen is overzealous when creating re-exports. As these imports are not used in the stubs, they shouldn't be here. Re-exporting them is more than likely just an artifact of the implementation. Same for serialize.pyi.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for sharing that! makes a lot of sense.

@srittau srittau merged commit ce94e8b into python:master Jul 12, 2021
@exemplary-citizen exemplary-citizen deleted the slumber branch July 12, 2021 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants