Skip to content
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

["Request"] API docs are missing for various apis #3044

Closed
myuwono opened this issue Apr 24, 2023 · 3 comments · Fixed by #3082
Closed

["Request"] API docs are missing for various apis #3044

myuwono opened this issue Apr 24, 2023 · 3 comments · Fixed by #3082

Comments

@myuwono
Copy link
Collaborator

myuwono commented Apr 24, 2023

What version are you currently using?
1.2.0-RC

What would you like to see?

User that is new to the library can benefit from a better API documentation. The api docs can be improved by providing description, use cases, and example code. While some API docs are really well documented, some functions were missing documentation.

These are the api docs, for instance can be improved:

@nomisRev
Copy link
Member

Thank you for the issue @myuwono! 🙏

@myuwono
Copy link
Collaborator Author

myuwono commented Apr 24, 2023

Thank you @nomisRev! @jbunton-atlassian kindly pointed it out for us. I just raised an issue to track it on his behalf. I believe it is a small gap that can be addressed to help improve adoption.

Kudos to Arrow team, thank you for all the amazing things and kindly listening to our voices, simplifying the library and building such an amazing community.

The API surface has shrunk significantly with the simplifications, so it seems relatively manageable. So I'm also keen to help a few. Keen to approach this incrementally. I can see #3038 which is a doco for ensure. I'll try raising one or two kdocs on perhaps either builder or option builder this week..

@myuwono
Copy link
Collaborator Author

myuwono commented Apr 24, 2023

Looks like a duplicate of #3037

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 a pull request may close this issue.

2 participants