-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
buffer: add JSDoc to blob bytes method #54117
buffer: add JSDoc to blob bytes method #54117
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #54117 +/- ##
==========================================
- Coverage 87.08% 87.07% -0.02%
==========================================
Files 643 643
Lines 181581 181584 +3
Branches 34897 34885 -12
==========================================
- Hits 158122 158106 -16
- Misses 16747 16754 +7
- Partials 6712 6724 +12
|
033f174
to
db5b2a2
Compare
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 with a nit: the commit message needs to be fixed before this PR can land, please read the guideline here
My suggested commit message is: buffer: add JSDoc to blob bytes method
, you can amend the commit and do a force push.
db5b2a2
to
50f9bde
Compare
@jakecastelli, ok thanks a lot for the guidance, I changed the commit message and now should be ok, if there is something else to fix let me know and I'll fix it asap, thanks again :) |
Landed in a3cd290 |
PR-URL: #54117 Reviewed-By: Jake Yuesong Li <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
PR-URL: #54117 Reviewed-By: Jake Yuesong Li <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
added JSDoc comment that describes the return type of the types blob method.
This fixes part of: #54105