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

Initial JSDoc #41

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Initial JSDoc #41

wants to merge 4 commits into from

Conversation

petrspirka
Copy link
Contributor

Adds initial JSDoc. This PR is meant more as an initial draft and is expected to receive more commits which will address inconsistent/incorrect documentation before merging into the base repository.

Added basic JSDoc that covers most public/protected methods and exports. This commit is intended more as a draft which should first be checked for inconsistencies.
Removed all inconsistent ID usages and old information
@Thomasparsley
Copy link
Owner

If this PR is still not ready for merging or review, please convert it into a draft.

@petrspirka petrspirka marked this pull request as draft September 18, 2023 06:02
@slunimara slunimara self-requested a review September 20, 2023 11:45
@petrspirka
Copy link
Contributor Author

I've checked over the documentation and I believe that no changes will be necessary after all (or at least I didn't find any) so I'll mark the draft as a normal PR now.

@petrspirka petrspirka marked this pull request as ready for review September 23, 2023 08:59
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.

None yet

2 participants