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.

Sign up for GitHub

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

Jump to bottom

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
Merged

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 1 commit into from
Dec 11, 2023

Conversation

nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot nodejs-github-bot commented Oct 29, 2023 •
edited by aduh95
Loading

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

CI: https://ci.nodejs.org/job/node-test-pull-request/55304/

@nodejs-github-bot
Copy link
Collaborator Author

CI: https://ci.nodejs.org/job/node-test-pull-request/55306/

MoLow
MoLow previously approved these changes Oct 29, 2023
View reviewed changes
@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.

@github-actions github-actions bot mentioned this pull request Oct 30, 2023
Open
25 tasks
@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.

@github-actions github-actions bot mentioned this pull request Oct 31, 2023
Open
17 tasks
@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 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

CI: https://ci.nodejs.org/job/node-test-pull-request/56213/

@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
@nodejs-github-bot @RafaelGSS
tools: update doc to highlight.js@11.9.0 unified@11.0.4
297cb6f 
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
Merged
richardlau pushed a commit that referenced this pull request Mar 25, 2024
@nodejs-github-bot @richardlau
tools: update doc to highlight.js@11.9.0 unified@11.0.4
c7350c2 
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
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aduh95 aduh95 aduh95 approved these changes

@aymen94 aymen94 aymen94 approved these changes

@jasnell jasnell jasnell approved these changes

@MoLow MoLow MoLow left review comments

Assignees
No one assigned
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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@nodejs-github-bot @lpinca @aduh95 @richardlau @jasnell @MoLow @aymen94