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: update tools .md files for upcoming lint/formatting #40155

Merged
merged 1 commit into from
Sep 21, 2021

Conversation

Trott
Copy link
Member

@Trott Trott commented Sep 19, 2021

Looks like only one .md file in tools needs to be changed.

@Trott Trott requested a review from aduh95 September 19, 2021 19:50
@nodejs-github-bot nodejs-github-bot added i18n-api Issues and PRs related to the i18n implementation. icu Issues and PRs related to the ICU dependency. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. labels Sep 19, 2021
@Trott Trott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 20, 2021
@Trott Trott added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 21, 2021
@github-actions github-actions 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 Sep 21, 2021
@github-actions
Copy link
Contributor

Commit Queue failed
- Loading data for nodejs/node/pull/40155
✔  Done loading data for nodejs/node/pull/40155
----------------------------------- PR info ------------------------------------
Title      doc: update tools .md files for upcoming lint/formatting (#40155)
Author     Rich Trott  (@Trott)
Branch     Trott:tools-md -> nodejs:master
Labels     tools, i18n-api, author ready, needs-ci, icu
Commits    1
 - doc: update tools .md files for upcoming lint/formatting
Committers 1
 - Rich Trott 
PR-URL: https://github.com/nodejs/node/pull/40155
Reviewed-By: Antoine du Hamel 
Reviewed-By: Michaël Zasso 
Reviewed-By: James M Snell 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40155
Reviewed-By: Antoine du Hamel 
Reviewed-By: Michaël Zasso 
Reviewed-By: James M Snell 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sun, 19 Sep 2021 19:50:26 GMT
   ✔  Approvals: 3
   ✔  - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/40155#pullrequestreview-758189003
   ✔  - Michaël Zasso (@targos) (TSC): https://github.com/nodejs/node/pull/40155#pullrequestreview-758347359
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/40155#pullrequestreview-758696742
   ✔  Last GitHub Actions 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/1259413001

Looks like only one .md file in `tools` needs to be changed.

PR-URL: nodejs#40155
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@Trott
Copy link
Member Author

Trott commented Sep 21, 2021

Landed in 0c95dea

@Trott Trott merged commit 0c95dea into nodejs:master Sep 21, 2021
@Trott Trott deleted the tools-md branch September 21, 2021 23:59
targos pushed a commit that referenced this pull request Oct 4, 2021
Looks like only one .md file in `tools` needs to be changed.

PR-URL: #40155
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue-failed An error occurred while landing this pull request using GitHub Actions. i18n-api Issues and PRs related to the i18n implementation. icu Issues and PRs related to the ICU dependency. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants