Skip to content

doc,test: update v8 method doc and comment - #35795

Merged
Trott merged 1 commit into
nodejs:masterfrom
Trott:cachedDataVersionTag
Oct 27, 2020
Merged

doc,test: update v8 method doc and comment#35795
Trott merged 1 commit into
nodejs:masterfrom
Trott:cachedDataVersionTag

Conversation

@Trott

Copy link
Copy Markdown
Member

Update documentation and test comment for v8.cachedDataVersionTag().

Checklist

@Trott
Trottforce-pushed the cachedDataVersionTag branch 2 times, most recently from 9859b08 to ad97155CompareOctober 25, 2020 12:59
@TrottTrott added request-ci Add this label to start a Jenkins CI on a PR. v8 module Issues and PRs related to the "v8" subsystem. labels Oct 25, 2020
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 25, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@codecov-io

codecov-io commented Oct 25, 2020

Copy link
Copy Markdown

Codecov Report

Merging #35795 into master will decrease coverage by 8.51%.
The diff coverage is 81.34%.

@@ Coverage Diff @@## master #35795 +/- ##
==========================================
- Coverage 96.40% 87.89% -8.52% 
==========================================
Files 223 477 +254 Lines 73685 113172 +39487 Branches 0 25426 +25426 ==========================================
+ Hits 71038 99473 +28435 - Misses 2647 7996 +5349 - Partials 0 5703 +5703 
Impacted FilesCoverage Δ
src/node_http2.h92.06% <ø> (ø)
src/node_report.cc90.71% <ø> (ø)
src/inspector_profiler.cc76.17% <69.44%> (ø)
lib/internal/dns/utils.js98.44% <75.00%> (-1.56%)⬇️
src/cares_wrap.cc63.42% <77.77%> (ø)
src/node_http2.cc83.76% <81.25%> (ø)
lib/internal/dns/promises.js100.00% <100.00%> (ø)
lib/internal/http2/core.js94.97% <100.00%> (-1.98%)⬇️
lib/internal/streams/destroy.js94.66% <100.00%> (-2.19%)⬇️
lib/internal/streams/from.js98.01% <100.00%> (ø)
... and 399 more

Comment threadtest/parallel/test-v8-version-tag.js Outdated
@Trott
Trottforce-pushed the cachedDataVersionTag branch from ad97155 to 641ef5aCompareOctober 26, 2020 04:29
@TrottTrott added the request-ci Add this label to start a Jenkins CI on a PR. label Oct 26, 2020
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 26, 2020
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

nodejs-github-bot commented Oct 26, 2020

Copy link
Copy Markdown
Collaborator

Update documentation and test comment for v8.cachedDataVersionTag().
PR-URL: nodejs#35795
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@Trott
Trottforce-pushed the cachedDataVersionTag branch from 641ef5a to 245ec6fCompareOctober 27, 2020 13:19
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in 245ec6f

@Trott
Trott merged commit 245ec6f into nodejs:masterOct 27, 2020
@Trott
Trott deleted the cachedDataVersionTag branch October 27, 2020 13:20
targos pushed a commit that referenced this pull request Nov 3, 2020
Update documentation and test comment for v8.cachedDataVersionTag().
PR-URL: #35795
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@targostargos mentioned this pull request Nov 3, 2020
BethGriggs pushed a commit that referenced this pull request Dec 8, 2020
Update documentation and test comment for v8.cachedDataVersionTag().
PR-URL: #35795
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
BethGriggs pushed a commit that referenced this pull request Dec 10, 2020
Update documentation and test comment for v8.cachedDataVersionTag().
PR-URL: #35795
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@BethGriggsBethGriggs mentioned this pull request Dec 10, 2020
BethGriggs pushed a commit that referenced this pull request Dec 15, 2020
Update documentation and test comment for v8.cachedDataVersionTag().
PR-URL: #35795
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v8 moduleIssues and PRs related to the "v8" subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@Trott@nodejs-github-bot@codecov-io@lpinca@watilde@gireeshpunathil