Skip to content

doc: fix YAML syntax errors#35529

Closed
aduh95 wants to merge 1 commit into
nodejs:masterfrom
aduh95:fix-build_docs-ci
Closed

doc: fix YAML syntax errors#35529
aduh95 wants to merge 1 commit into
nodejs:masterfrom
aduh95:fix-build_docs-ci

Conversation

@aduh95
Copy link
Copy Markdown
Contributor

@aduh95 aduh95 commented Oct 6, 2020

It seems v12.19.0 release commit contains an unintended YAML syntax error, which causes CI to fail for every PR.

cc @codebytere

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. zlib Issues and PRs related to the zlib subsystem. labels Oct 6, 2020
@jasnell jasnell added the fast-track PRs that do not need to wait for 72 hours to land. label Oct 6, 2020
Copy link
Copy Markdown
Member

@jasnell jasnell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please 👍🏻 to fast-track

@jasnell
Copy link
Copy Markdown
Member

jasnell commented Oct 6, 2020

looks like there may be another one in there...

image

@aduh95 aduh95 force-pushed the fix-build_docs-ci branch from 388d528 to d1044ff Compare October 6, 2020 21:31
@aduh95 aduh95 changed the title doc: fix YAML in zlib.md doc: fix YAML syntax errors Oct 6, 2020
@aduh95 aduh95 force-pushed the fix-build_docs-ci branch from d1044ff to ff5e903 Compare October 6, 2020 21:34
@aduh95
Copy link
Copy Markdown
Contributor Author

aduh95 commented Oct 6, 2020

OK, build-docs now passes, PTAL.

@jasnell jasnell added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 6, 2020
@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 Oct 6, 2020
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 6, 2020

Commit Queue failed
- Loading data for nodejs/node/pull/35529
✔  Done loading data for nodejs/node/pull/35529
----------------------------------- PR info ------------------------------------
Title      doc: fix YAML syntax errors (#35529)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     aduh95:fix-build_docs-ci -> nodejs:master
Labels     doc, fast-track, zlib
Commits    1
 - doc: fix YAML syntax errors
Committers 1
 - Antoine du Hamel 
PR-URL: https://github.com/nodejs/node/pull/35529
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/35529
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
--------------------------------------------------------------------------------
   ✖  GitHub CI is still running
   ℹ  Doc-only changes
   ℹ  This PR was created on Tue, 06 Oct 2020 21:26:48 GMT
   ✔  Approvals: 2
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/35529#pullrequestreview-503377599
   ✔  - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/35529#pullrequestreview-503372496
   ℹ  This PR is being fast-tracked
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
@jasnell
Copy link
Copy Markdown
Member

jasnell commented Oct 6, 2020

Will land as soon as the CI finishes up

@jasnell jasnell removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Oct 6, 2020
@aduh95
Copy link
Copy Markdown
Contributor Author

aduh95 commented Oct 6, 2020

CI is red on test-linux-with-quic: test-net-listen-ipv6only and green otherwise.

jasnell pushed a commit that referenced this pull request Oct 6, 2020
PR-URL: #35529
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Beth Griggs <bgriggs@redhat.com>
@jasnell
Copy link
Copy Markdown
Member

jasnell commented Oct 6, 2020

Landed in ad012c9

@jasnell jasnell closed this Oct 6, 2020
@aduh95 aduh95 deleted the fix-build_docs-ci branch October 6, 2020 22:54
BethGriggs pushed a commit that referenced this pull request Oct 13, 2020
PR-URL: #35529
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Beth Griggs <bgriggs@redhat.com>
@MylesBorins MylesBorins mentioned this pull request Oct 14, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
PR-URL: nodejs#35529
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Beth Griggs <bgriggs@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 72 hours to land. zlib Issues and PRs related to the zlib subsystem.

6 participants