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

Add code documentation for process() #47

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

francisohara24
Copy link

I added docstrings and type annotations for the process() function to make it easier for users to figure out how exactly the package can be used to extract text and images.
Although documentation is provided in the README.md file in the package's GitHub repository, I think it's important that the code itself is documented as well so that users don't have to visit the repository to figure out how to use the package, which is the situation I found myself in.
It'll also help IDEs do code completion for developers using the package.

add docstring and type annotations for `process()` function to describe what it does and how it works since this is the main function users will be interacting with.
add docstring and type annotations for `process()` function to describe what it does and how it works since this is the main function users will be interacting with. Update with examples in docstring.
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.

1 participant