Skip to content
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

doc: fix errors in Performance hooks doc #43152

Merged
merged 1 commit into from
May 20, 2022
Merged

Conversation

y1d7ng
Copy link
Contributor

@y1d7ng y1d7ng commented May 19, 2022

Screen Shot 2022-05-20 at 12 01 02 AM

@nodejs-github-bot 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 Trott added the fast-track PRs that do not need to wait for 48 hours to land. label May 19, 2022
@github-actions
Copy link
Contributor

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

@Trott Trott added the commit-queue Add this label to land a pull request using GitHub Actions. label May 19, 2022
@nodejs-github-bot 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
@nodejs-github-bot
Copy link
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 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 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
@nodejs-github-bot
Copy link
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 <[email protected]>
Reviewed-By: Zeyu "Alex" Yang <[email protected]>
@Trott
Copy link
Member

Trott commented May 20, 2022

Landed in 38f4810

@Trott Trott merged commit 38f4810 into nodejs:master May 20, 2022
@y1d7ng 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 <[email protected]>
Reviewed-By: Zeyu "Alex" Yang <[email protected]>
@bengl 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 <[email protected]>
Reviewed-By: Zeyu "Alex" Yang <[email protected]>
danielleadams pushed a commit that referenced this pull request Jun 27, 2022
PR-URL: #43152
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Zeyu "Alex" Yang <[email protected]>
targos pushed a commit that referenced this pull request Jul 12, 2022
PR-URL: #43152
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Zeyu "Alex" Yang <[email protected]>
targos pushed a commit that referenced this pull request Jul 31, 2022
PR-URL: #43152
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Zeyu "Alex" Yang <[email protected]>
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 <[email protected]>
Reviewed-By: Zeyu "Alex" Yang <[email protected]>
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