-
Notifications
You must be signed in to change notification settings - Fork 629
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(semver): improve docs #4846
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4846 +/- ##
==========================================
- Coverage 92.35% 92.35% -0.01%
==========================================
Files 487 487
Lines 41531 41539 +8
Branches 5405 5405
==========================================
+ Hits 38358 38364 +6
- Misses 3117 3119 +2
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.
Great to have this done! My main nit is the proper casing of "SemVer".
semver/range_max.ts
Outdated
* | ||
* @example Usage | ||
* ```ts | ||
* // This API is deprecated |
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.
Perhaps, we can have two snippets in this example; one for the deprecated usage and one for the new API's usage, like a migration guide.
Co-authored-by: Asher Gomez <ashersaupingomez@gmail.com>
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.
Nice!
part of #3764
This improves the API docs of
@std/semver