Skip to content

docs(tracer): review API docs & README for Tracer #2992

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

Merged
merged 4 commits into from
Aug 29, 2024
Merged

Conversation

am29d
Copy link
Contributor

@am29d am29d commented Aug 28, 2024

Summary

Changes

Please provide a summary of what's being changed

This PR overhauls the JSDoc comments on each type, function, class, and method of the parameters utility so that it provides better and relevant info to customers using the API docs or wanting to get contextual info within their IDE.

Please add the issue number below, if no issue is present the PR might get blocked and not be reviewed

Issue number: closes #2376


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@am29d am29d requested a review from a team August 28, 2024 14:39
@am29d am29d requested a review from a team as a code owner August 28, 2024 14:39
@boring-cyborg boring-cyborg bot added internal PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.) tracer This item relates to the Tracer Utility labels Aug 28, 2024
@pull-request-size pull-request-size bot added the size/L PRs between 100-499 LOC label Aug 28, 2024
@am29d am29d changed the title change readme to common structure docs(tracer): review API docs & README for Logger Aug 28, 2024
@am29d am29d marked this pull request as draft August 28, 2024 16:21
@am29d am29d self-assigned this Aug 28, 2024
@am29d am29d changed the title docs(tracer): review API docs & README for Logger docs(tracer): review API docs & README for Tracers Aug 29, 2024
@am29d am29d changed the title docs(tracer): review API docs & README for Tracers docs(tracer): review API docs & README for Tracer Aug 29, 2024
Copy link

@am29d am29d marked this pull request as ready for review August 29, 2024 12:19
@dreamorosi dreamorosi merged commit 1bab6ea into main Aug 29, 2024
11 checks passed
@dreamorosi dreamorosi deleted the am29d/issue2376 branch August 29, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.) size/L PRs between 100-499 LOC tracer This item relates to the Tracer Utility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Maintenance: review API docs & README for Tracer
2 participants