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

tools: update doc to highlight.js@11.9.0 remark-frontmatter@5.0.0 remark-gfm@4.0.0 remark-html@16.0.1 remark-parse@11.0.0 remark-rehype@11.0.0 unified@11.0.4 unist-util-select@5.1.0 #50459

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot nodejs-github-bot commented Oct 29, 2023

This is an automated update of doc to highlight.js@11.9.0 remark-frontmatter@5.0.0 remark-gfm@4.0.0 remark-html@16.0.1 remark-parse@11.0.0 remark-rehype@11.0.0 (updating those breaks the no-intl CI) unified@11.0.4 unist-util-select@5.1.0.

@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Oct 29, 2023
@nodejs-github-bot
Copy link
Collaborator Author

Review requested:

  • @nodejs/web-infra

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Oct 29, 2023
@richardlau richardlau added the request-ci Add this label to start a Jenkins CI on a PR. label Oct 29, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 29, 2023
@nodejs-github-bot
Copy link
Collaborator Author

@nodejs-github-bot
Copy link
Collaborator Author

MoLow
MoLow previously approved these changes Oct 29, 2023
@lpinca lpinca added the blocked PRs that are blocked by other issues or PRs. label Oct 29, 2023
@lpinca
Copy link
Member

lpinca commented Oct 29, 2023

See #50414.

@aduh95
Copy link
Contributor

aduh95 commented Oct 30, 2023

I think the logical fix is to build the docs with a stable node binary on the without-intl job, so we don't crash the build if the deps uses a regex with the u flag. @nodejs/build how can we make this happen? IIRC the setup job is not available publicly, only a build member can do this sort of change.

@richardlau
Copy link
Member

@aduh95 FWIW the without-intl job runs:

NODE_TEST_DIR=/home/iojs/node-tmp FLAKY_TESTS=dontcare CONFIG_FLAGS=' --without-intl' make run-ci -j 4

Given that we describe how to build without intl in https://github.com/nodejs/node/blob/main/BUILDING.md#building-without-intl-support I'd prefer if we had a solution that worked without involving changes to the Jenkins job -- i.e. running the Makefile after configure --without-intl should just work.

@nodejs-github-bot nodejs-github-bot changed the title tools: update doc to highlight.js@11.9.0 remark-frontmatter@5.0.0 remark-gfm@4.0.0 remark-html@16.0.1 remark-parse@11.0.0 remark-rehype@11.0.0 unified@11.0.4 tools: update doc to highlight.js@11.9.0 remark-frontmatter@5.0.0 remark-gfm@4.0.0 remark-html@16.0.1 remark-parse@11.0.0 remark-rehype@11.0.0 unified@11.0.4 unist-util-select@5.1.0 Nov 12, 2023
@aduh95 aduh95 force-pushed the actions/tools-update-doc branch from 002a698 to 88b53dd Compare December 10, 2023 23:38
@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. and removed blocked PRs that are blocked by other issues or PRs. labels Dec 10, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 10, 2023
@nodejs-github-bot
Copy link
Collaborator Author

@aduh95 aduh95 merged commit ab93a35 into main Dec 11, 2023
46 of 50 checks passed
@aduh95 aduh95 deleted the actions/tools-update-doc branch December 11, 2023 00:41
@aduh95
Copy link
Contributor

aduh95 commented Dec 11, 2023

Landed in ab93a35

RafaelGSS pushed a commit that referenced this pull request Dec 15, 2023
And unist-util-select@5.1.0.

PR-URL: #50459
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
@RafaelGSS RafaelGSS mentioned this pull request Dec 15, 2023
richardlau pushed a commit that referenced this pull request Mar 25, 2024
And unist-util-select@5.1.0.

PR-URL: #50459
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
@richardlau richardlau mentioned this pull request Mar 25, 2024
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. doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants