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: remove the last <pre> tag #53741

Merged
merged 2 commits into from
Jul 6, 2024
Merged

doc: remove the last <pre> tag #53741

merged 2 commits into from
Jul 6, 2024

Conversation

ovflowd
Copy link
Member

@ovflowd ovflowd commented Jul 6, 2024

This PR removes the last <pre> tag from the API docs, which is needed in preparation for the new API Doc Tooling

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/loaders
@ovflowd ovflowd added the fast-track PRs that do not need to wait for 48 hours to land. label Jul 6, 2024
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. module Issues and PRs related to the module subsystem. labels Jul 6, 2024
@ovflowd ovflowd removed the module Issues and PRs related to the module subsystem. label Jul 6, 2024
Copy link
Contributor

github-actions bot commented Jul 6, 2024

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

doc/api/modules.md Outdated Show resolved Hide resolved
Co-authored-by: Aviv Keller <38299977+RedYetiDev@users.noreply.github.com>
@ovflowd ovflowd added commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. labels Jul 6, 2024
@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 Jul 6, 2024
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/53741
✔  Done loading data for nodejs/node/pull/53741
----------------------------------- PR info ------------------------------------
Title      doc: remove the last 
 tag (#53741)
Author     Claudio W  (@ovflowd)
Branch     ovflowd:doc-remove-pre-tag -> nodejs:main
Labels     doc, fast-track, commit-queue-squash
Commits    2
 - doc: remove the last 
 tag
 - Update doc/api/modules.md
Committers 1
 - GitHub 
PR-URL: https://github.com/nodejs/node/pull/53741
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Luigi Pinca 
Reviewed-By: Ulises Gascón 
Reviewed-By: Marco Ippolito 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/53741
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Luigi Pinca 
Reviewed-By: Ulises Gascón 
Reviewed-By: Marco Ippolito 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sat, 06 Jul 2024 00:15:46 GMT
   ✔  Approvals: 4
   ✔  - Yagiz Nizipli (@anonrig): https://github.com/nodejs/node/pull/53741#pullrequestreview-2161354259
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/53741#pullrequestreview-2161445675
   ✔  - Ulises Gascón (@UlisesGascon): https://github.com/nodejs/node/pull/53741#pullrequestreview-2161501096
   ✔  - Marco Ippolito (@marco-ippolito) (TSC): https://github.com/nodejs/node/pull/53741#pullrequestreview-2161504423
   ℹ  This PR is being fast-tracked
   ✘  This PR needs to wait 37 more hours to land (or 0 hours if there is 1 more approval (👍) of the fast-track request from collaborators).
   ✔  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/9818600432
@ovflowd
Copy link
Member Author

ovflowd commented Jul 6, 2024

Can someone else approve the fast track? 👀

@aduh95 aduh95 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 Jul 6, 2024
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 6, 2024
@nodejs-github-bot nodejs-github-bot merged commit ea71187 into main Jul 6, 2024
25 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in ea71187

@nodejs-github-bot nodejs-github-bot deleted the doc-remove-pre-tag branch July 6, 2024 11:06
@RedYetiDev RedYetiDev added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jul 6, 2024
@RedYetiDev
Copy link
Member

RedYetiDev commented Jul 6, 2024

(Post-Mortem label fix - author ready w/ commit-queue, right?)

aduh95 pushed a commit that referenced this pull request Jul 12, 2024
PR-URL: #53741
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
@aduh95 aduh95 mentioned this pull request Jul 12, 2024
aduh95 pushed a commit that referenced this pull request Jul 16, 2024
PR-URL: #53741
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@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-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.
8 participants