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: fix YAML syntax errors #35529

Closed
wants to merge 1 commit into from
Closed

Conversation

aduh95
Copy link
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 48 hours to land. label Oct 6, 2020
Copy link
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
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
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
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
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
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
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 48 hours to land. zlib Issues and PRs related to the zlib subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants