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 Documentation Links #5

Merged
merged 2 commits into from
Jul 26, 2023
Merged

Add Documentation Links #5

merged 2 commits into from
Jul 26, 2023

Conversation

jeremytwfortune
Copy link
Collaborator

Description of Changes

Updates docstrings/jsdoc to point to point directly to API docs.

Issue Link

This PR addresses the following issues:

Security

REMINDER: All file contents are public.

  • I have ensured no secure credentials or sensitive information remain in code, metadata, comments, etc. Of particular note: No temporary testing changes committed such as API base URLs, access tokens, print/log statements, etc.
  • My changes do not introduce any security risks, or any such risks have been properly mitigated.

This PR does not change any code, only documentation.

Reviewers

  • I have assigned the appropriate reviewer(s).

Minimally, a second set of eyes is needed ensure no non-public information is published. Consider also including subject-matter experts and editing/style reviewers.

@jeremytwfortune jeremytwfortune added the documentation Improvements or additions to documentation label Jul 24, 2023
@jeremytwfortune jeremytwfortune requested a review from amarsan July 24, 2023 18:58
@jeremytwfortune jeremytwfortune mentioned this pull request Jul 24, 2023
@jeremytwfortune jeremytwfortune merged commit bb7227d into main Jul 26, 2023
@jeremytwfortune jeremytwfortune deleted the direct-doc-links branch July 26, 2023 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants