GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — doc: fix YAML lint error on master by Trott · Pull Request #35709 · nodejs/node

Conversation

@Trott

@Trott Trott commented Oct 19, 2020

Copy link
Copy Markdown
Member

Refs: https://github.com/nodejs/node/runs/1275049330

Checklist
  • documentation is changed or added
  • commit message follows commit guidelines

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to Node.js documentation. process Issues and PRs related to the process subsystem. labels Oct 19, 2020
@Trott

Trott commented Oct 19, 2020 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

Fast-track to fix make lint on master? Please 👍 here.
targos and aduh95 reacted with thumbs up emoji
All reactions
  • 👍 2 reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@Trott
Trott requested review from aduh95 and gireeshpunathil October 19, 2020 12:45
@Trott Trott added the fast-track PRs proposed for a shorter-than-standard waiting period before landing. label Oct 19, 2020
targos
targos approved these changes Oct 19, 2020
View reviewed changes
aduh95
aduh95 approved these changes Oct 19, 2020
View reviewed changes
@Trott
doc: fix YAML lint error on master
1103b15 
Refs: https://github.com/nodejs/node/runs/1275049330

PR-URL: nodejs#35709
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@Trott
Trott force-pushed the fixup-lint branch from 20d1295 to 1103b15 Compare October 19, 2020 12:54
@Trott
Trott merged commit 1103b15 into nodejs:master Oct 19, 2020
@Trott

Trott commented Oct 19, 2020

Copy link
Copy Markdown
Member Author

Landed in 1103b15
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@Trott
Trott deleted the fixup-lint branch October 19, 2020 12:55
BethGriggs pushed a commit that referenced this pull request Dec 8, 2020
@Trott @BethGriggs
doc: fix YAML lint error on master
59b7583 
Refs: https://github.com/nodejs/node/runs/1275049330

PR-URL: #35709
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
targos pushed a commit that referenced this pull request Mar 3, 2021
@Trott @targos
doc: fix YAML lint error on master
44d6862 
Refs: https://github.com/nodejs/node/runs/1275049330

PR-URL: #35709
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
MylesBorins pushed a commit that referenced this pull request Apr 6, 2021
@Trott @MylesBorins
doc: fix YAML lint error on master
f41c28c 
Refs: https://github.com/nodejs/node/runs/1275049330

PR-URL: #35709
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@danielleadams danielleadams mentioned this pull request May 3, 2021
v14.17.0 proposal #38507
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
@gireeshpunathil gireeshpunathil Awaiting requested review from gireeshpunathil
+1 more reviewer
@targos targos targos approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

doc Issues and PRs related to Node.js documentation. fast-track PRs proposed for a shorter-than-standard waiting period before landing. process Issues and PRs related to the process subsystem.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@Trott @targos @aduh95 @nodejs-github-bot
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.