-
Notifications
You must be signed in to change notification settings - Fork 630
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
docs(uuid): update module docs #4790
Conversation
marvinhagemeister
commented
May 21, 2024
- Some wording changes
- Show an example of how to use this module on the main page on jsr.io
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4790 +/- ##
=======================================
Coverage 92.35% 92.36%
=======================================
Files 488 488
Lines 41581 41604 +23
Branches 5405 5405
=======================================
+ Hits 38403 38426 +23
Misses 3122 3122
Partials 56 56 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PTAL at https://github.com/denoland/deno_std/blob/main/.github/CONTRIBUTING.md#documentation
It contains steps such as adding uuid/mod.ts
to the lint:docs
task.
Co-authored-by: Asher Gomez <ashersaupingomez@gmail.com>
_tools/check_docs.ts
Outdated
"../uuid/v1.ts", | ||
"../uuid/v3.ts", | ||
"../uuid/v4.ts", | ||
"../uuid/v5.ts", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Files are done individually because deno doc --lint uuid/mod.ts
doesn't work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM