FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: fix errors in Performance hooks doc by Y1D7NG · Pull Request #43152 · nodejs/node · GitHub

/ node Public

doc: fix errors in Performance hooks doc - #43152

Merged
Trott merged 1 commit into
nodejs:masterfrom
Y1D7NG:doc-perf-hooks
May 20, 2022
Merged

doc: fix errors in Performance hooks doc#43152
Trott merged 1 commit into
nodejs:masterfrom
Y1D7NG:doc-perf-hooks

Conversation

Y1D7NG commented May 19, 2022

Copy link
Copy Markdown
Contributor

nodejs-github-bot added doc Issues and PRs related to the documentations. perf_hooks Issues and PRs related to the implementation of the Performance Timing API. labels May 19, 2022
Trott added the fast-track PRs that do not need to wait for 48 hours to land. label May 19, 2022

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @Trott. Please 👍 to approve.

Trott added the commit-queue Add this label to land a pull request using GitHub Actions. label May 19, 2022
nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels May 19, 2022

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/43152
✔  Done loading data for nodejs/node/pull/43152
----------------------------------- PR info ------------------------------------
Title      doc: fix errors in Performance hooks doc (#43152)
Author     OneNail  (@OneNail)
Branch     OneNail:doc-perf-hooks -> nodejs:master
Labels     doc, fast-track, perf_hooks
Commits    1
 - doc: fix errors in Performance hooks doc
Committers 1
 - OneNail 
PR-URL: https://github.com/nodejs/node/pull/43152
Reviewed-By: Rich Trott 
Reviewed-By: Zeyu "Alex" Yang 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/43152
Reviewed-By: Rich Trott 
Reviewed-By: Zeyu "Alex" Yang 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 19 May 2022 16:02:58 GMT
   ✔  Approvals: 2
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/43152#pullrequestreview-979188144
   ✔  - Zeyu "Alex" Yang (@himself65): https://github.com/nodejs/node/pull/43152#pullrequestreview-979343100
   ℹ  This PR is being fast-tracked
   ✖  The fast-track request requires at least one collaborator's approval (👍).
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/2355056650

Trott added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels May 20, 2022
nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels May 20, 2022

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/43152
✔  Done loading data for nodejs/node/pull/43152
----------------------------------- PR info ------------------------------------
Title      doc: fix errors in Performance hooks doc (#43152)
Author     OneNail  (@OneNail)
Branch     OneNail:doc-perf-hooks -> nodejs:master
Labels     doc, fast-track, perf_hooks
Commits    1
 - doc: fix errors in Performance hooks doc
Committers 1
 - OneNail 
PR-URL: https://github.com/nodejs/node/pull/43152
Reviewed-By: Rich Trott 
Reviewed-By: Zeyu "Alex" Yang 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/43152
Reviewed-By: Rich Trott 
Reviewed-By: Zeyu "Alex" Yang 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 19 May 2022 16:02:58 GMT
   ✔  Approvals: 2
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/43152#pullrequestreview-979188144
   ✔  - Zeyu "Alex" Yang (@himself65): https://github.com/nodejs/node/pull/43152#pullrequestreview-979343100
   ℹ  This PR is being fast-tracked
   ✖  The fast-track request requires at least one collaborator's approval (👍).
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/2355654087

PR-URL: nodejs#43152
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zeyu "Alex" Yang <himself65@outlook.com>
Trott force-pushed the doc-perf-hooks branch from 8f94c95 to 38f4810 Compare May 20, 2022 02:34

Trott commented May 20, 2022

Copy link
Copy Markdown
Member

Landed in 38f4810

Trott merged commit 38f4810 into nodejs:master May 20, 2022
Y1D7NG deleted the doc-perf-hooks branch May 20, 2022 04:30
bengl pushed a commit that referenced this pull request May 30, 2022
PR-URL: #43152
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zeyu "Alex" Yang <himself65@outlook.com>
bengl mentioned this pull request May 31, 2022
juanarbol pushed a commit that referenced this pull request May 31, 2022
PR-URL: #43152
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zeyu "Alex" Yang <himself65@outlook.com>
danielleadams pushed a commit that referenced this pull request Jun 27, 2022
PR-URL: #43152
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zeyu "Alex" Yang <himself65@outlook.com>
targos pushed a commit that referenced this pull request Jul 12, 2022
PR-URL: #43152
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zeyu "Alex" Yang <himself65@outlook.com>
targos pushed a commit that referenced this pull request Jul 31, 2022
PR-URL: #43152
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zeyu "Alex" Yang <himself65@outlook.com>
guangwong pushed a commit to noslate-project/node that referenced this pull request Oct 10, 2022
PR-URL: nodejs/node#43152
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zeyu "Alex" Yang <himself65@outlook.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit-queue-failed An error occurred while landing this pull request using GitHub Actions. doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land. perf_hooks Issues and PRs related to the implementation of the Performance Timing API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL