Skip to content

tools: fix typo in tools/code_cache/README.md - #41657

Merged
aduh95 merged 1 commit into
nodejs:masterfrom
tniessen:tools-typo-node-cache-readme
Jan 23, 2022
Merged

tools: fix typo in tools/code_cache/README.md#41657
aduh95 merged 1 commit into
nodejs:masterfrom
tniessen:tools-typo-node-cache-readme

Conversation

@tniessen

Copy link
Copy Markdown
Member

No description provided.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/startup

@nodejs-github-bot nodejs-github-bot added the needs-ci PRs that need a full CI run. label Jan 23, 2022
@tniessen tniessen added fast-track PRs proposed for a shorter-than-standard waiting period before landing. tools Issues and PRs related to the tools directory. labels Jan 23, 2022
@github-actions

Copy link
Copy Markdown
Contributor

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

@VoltrexKeyva VoltrexKeyva removed the needs-ci PRs that need a full CI run. label Jan 23, 2022
@tniessen tniessen added the commit-queue PRs queued for automated landing through the Commit Queue. label Jan 23, 2022
@nodejs-github-bot nodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Jan 23, 2022
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/41657
✔  Done loading data for nodejs/node/pull/41657
----------------------------------- PR info ------------------------------------
Title      tools: fix typo in tools/node_cache/README.md (#41657)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     tniessen:tools-typo-node-cache-readme -> nodejs:master
Labels     tools, fast-track
Commits    1
 - tools: fix typo in tools/node_cache/README.md
Committers 1
 - GitHub 
PR-URL: https://github.com/nodejs/node/pull/41657
Reviewed-By: Richard Lau 
Reviewed-By: Rich Trott 
Reviewed-By: Colin Ihrig 
Reviewed-By: Mohammed Keyvanzadeh 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/41657
Reviewed-By: Richard Lau 
Reviewed-By: Rich Trott 
Reviewed-By: Colin Ihrig 
Reviewed-By: Mohammed Keyvanzadeh 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sun, 23 Jan 2022 00:20:24 GMT
   ✔  Approvals: 4
   ✔  - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/41657#pullrequestreview-860252933
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/41657#pullrequestreview-860253029
   ✔  - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/41657#pullrequestreview-860253133
   ✔  - Mohammed Keyvanzadeh (@VoltrexMaster): https://github.com/nodejs/node/pull/41657#pullrequestreview-860254632
   ℹ  This PR is being fast-tracked
   ✔  Last GitHub CI successful
   ✖  No Jenkins CI runs detected
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1734775034

@targos

targos commented Jan 23, 2022

Copy link
Copy Markdown
Member

node_cache -> code_cache

@tniessen
tniessen force-pushed the tools-typo-node-cache-readme branch from d63d8d9 to 8f2c28a Compare January 23, 2022 09:51
@tniessen tniessen changed the title tools: fix typo in tools/node_cache/README.md tools: fix typo in tools/code_cache/README.md Jan 23, 2022
@tniessen tniessen removed the commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. label Jan 23, 2022
@aduh95
aduh95 merged commit c18ca14 into nodejs:master Jan 23, 2022
@aduh95

aduh95 commented Jan 23, 2022

Copy link
Copy Markdown
Contributor

Landed in c18ca14

BethGriggs pushed a commit that referenced this pull request Jan 25, 2022
PR-URL: #41657
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
danielleadams pushed a commit that referenced this pull request Feb 28, 2022
PR-URL: #41657
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
danielleadams pushed a commit that referenced this pull request Mar 2, 2022
PR-URL: #41657
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
danielleadams pushed a commit that referenced this pull request Mar 3, 2022
PR-URL: #41657
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
danielleadams pushed a commit that referenced this pull request Mar 14, 2022
PR-URL: #41657
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track PRs proposed for a shorter-than-standard waiting period before landing. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants