Skip to content

tools: update doc to new version - #51192

Closed
nodejs-github-bot wants to merge 2 commits into
mainfrom
actions/tools-update-doc
Closed

tools: update doc to new version#51192
nodejs-github-bot wants to merge 2 commits into
mainfrom
actions/tools-update-doc

Conversation

@nodejs-github-bot

@nodejs-github-botnodejs-github-bot commented Dec 17, 2023

Copy link
Copy Markdown
Collaborator

This is an automated update of doc to new version.

@nodejs-github-botnodejs-github-bot added the tools Issues and PRs related to the tools directory. label Dec 17, 2023
@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

Review requested:

  • @nodejs/web-infra

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Dec 17, 2023
@marco-ippolito

Copy link
Copy Markdown
Member

I think this update should be split into several commits to prevent commit message lint failure

@aduh95aduh95 added the blocked PRs that are blocked by other issues or PRs. label Dec 17, 2023
@aduh95

Copy link
Copy Markdown
Contributor

This is blocked by micromark/micromark#157, see #49988 (comment)

@nodejs-github-botnodejs-github-bot changed the title tools: update doc to remark-frontmatter@5.0.0 remark-gfm@4.0.0 remark-html@16.0.1 remark-parse@11.0.0 remark-rehype@11.0.0tools: update doc to remark-frontmatter@5.0.0 remark-gfm@4.0.0 remark-html@16.0.1 remark-parse@11.0.0 remark-rehype@11.1.0Jan 14, 2024
@aduh95aduh95 added the needs-ci PRs that need a full CI run. label Feb 11, 2024
@MoLowMoLow added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 21, 2024
@MoLow

Copy link
Copy Markdown
Member

running a full CI cycle to check if this can get unblocked

@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 21, 2024
@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

@MoLow

Copy link
Copy Markdown
Member

still failing

10:25:08 export const unicodePunctuation = regexCheck(/\p{P}|\p{S}/u);
10:25:08 ^^^^^^^^^^^^^^
10:25:08 10:25:08 SyntaxError: Invalid regular expression: /\p{P}|\p{S}/u: Invalid property name
10:25:08 at compileSourceTextModule (node:internal/modules/esm/utils:337:16)
10:25:08 at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:168:18)
10:25:08 at callTranslator (node:internal/modules/esm/loader:428:14)
10:25:08 at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:434:30)

@marco-ippolito
marco-ippolitoforce-pushed the actions/tools-update-doc branch from 53c8f72 to fe425f3CompareMay 8, 2024 09:58
@marco-ippolito

Copy link
Copy Markdown
Member

amended commit otherwise we will never land this, I'll fix the tool in another PR

@nodejs-github-botnodejs-github-bot changed the title tools: update doc to remark-frontmatter@5.0.0 remark-gfm@4.0.0 remark-html@16.0.1 remark-parse@11.0.0 remark-rehype@11.1.0tools: update doc to new versionMay 12, 2024
@avivkelleravivkeller removed the blocked PRs that are blocked by other issues or PRs. label Sep 24, 2024
@avivkeller

Copy link
Copy Markdown
Member

(The blocking PR has been closed)

@aduh95aduh95 added the blocked PRs that are blocked by other issues or PRs. label Sep 24, 2024
@aduh95

Copy link
Copy Markdown
Contributor

(The blocking PR has been closed)

And the problem hasn't been addressed, the CI doesn't pass.

@aduh95aduh95 added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 14, 2024
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 14, 2024
@aduh95aduh95 added request-ci Add this label to start a Jenkins CI on a PR. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. and removed blocked PRs that are blocked by other issues or PRs. labels Nov 16, 2024
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 16, 2024
@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

@aduh95aduh95 added the commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. label Nov 16, 2024
@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

@codecov

codecovBot commented Nov 16, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.92%. Comparing base (b02cd41) to head (45e4f9c).
Report is 17 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #51192 +/- ##
==========================================
- Coverage 88.42% 87.92% -0.50% 
==========================================
Files 654 654 Lines 187852 187852 Branches 36134 35824 -310 ==========================================
- Hits 166102 165167 -935 - Misses 14989 15875 +886 - Partials 6761 6810 +49 

see 91 files with indirect coverage changes

@aduh95aduh95 added dont-land-on-v18.x dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. labels Nov 17, 2024
@marco-ippolitomarco-ippolito added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 17, 2024
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 17, 2024
@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor
Commit Queue failed
- Loading data for nodejs/node/pull/51192
✔ Done loading data for nodejs/node/pull/51192
----------------------------------- PR info ------------------------------------
Title tools: update doc to new version (#51192)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch nodejs-github-bot:actions/tools-update-doc -> nodejs:main
Labels doc, tools, author ready, needs-ci, commit-queue-rebase, dont-land-on-v18.x, dont-land-on-v20.x, dont-land-on-v22.x, dont-land-on-v23.x
Commits 2
- tools: disable doc building when ICU is not available
- tools: update doc to new version
Committers 2
- Antoine du Hamel <duhamelantoine1995@gmail.com>
- Node.js GitHub Bot <github-bot@iojs.org>
PR-URL: https://github.com/nodejs/node/pull/51192
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/51192
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
--------------------------------------------------------------------------------
⚠ Commits were pushed since the last approving review:
⚠ - tools: disable doc building when ICU is not available
⚠ - tools: update doc to new version
ℹ This PR was created on Sun, 17 Dec 2023 00:26:43 GMT
✔ Approvals: 4
✔ - Moshe Atlow (@MoLow) (TSC): https://github.com/nodejs/node/pull/51192#pullrequestreview-2203485934
✔ - Marco Ippolito (@marco-ippolito) (TSC): https://github.com/nodejs/node/pull/51192#pullrequestreview-2106067988
✔ - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/51192#pullrequestreview-2161903820
✔ - Ulises Gascón (@UlisesGascon): https://github.com/nodejs/node/pull/51192#pullrequestreview-2259349505
✔ Last GitHub CI successful
ℹ Last Full PR CI on 2024-11-16T23:04:18Z: https://ci.nodejs.org/job/node-test-pull-request/63590/
- Querying data for job/node-test-pull-request/63590/
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/11878353074

@nodejs-github-botnodejs-github-bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Nov 17, 2024
@marco-ippolitomarco-ippolito 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 Nov 17, 2024
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 17, 2024
@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

Landed in b0051de...b1ee186

nodejs-github-bot pushed a commit that referenced this pull request Nov 17, 2024
PR-URL: #51192
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
nodejs-github-bot added a commit that referenced this pull request Nov 17, 2024
PR-URL: #51192
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.commit-queue-rebaseAdd this label to allow the Commit Queue to land a PR in several commits.docIssues and PRs related to the documentations.dont-land-on-v22.xPRs that should not land on the v22.x-staging branch and should not be released in v22.x.needs-ciPRs that need a full CI run.toolsIssues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@nodejs-github-bot@marco-ippolito@aduh95@MoLow@avivkeller@jasnell@UlisesGascon