-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add basic docstrings for Application
and Request
#2626
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Application
and Request
@kmahar I'm assuming this needs an actual release so you can pull in the updated dependency with the docs? |
Original comment:
|
@0xTim Yeah, we would need a release. That said, I can work around it in the short term by manually editing the generated docs to add in the content, so it's not super urgent, just nice to have for the future 🙂 |
Cool I'll do a release. Yay for automation! |
These changes are now available in 4.45.5 |
Awesome, thank you! 🎉 |
This adds inline documentation for
Application
andRequest
. This helps when trying to integrate Jazzy (realm/jazzy#936)