Skip to content

Conversation

@cjihrig
Copy link
Contributor

Update ESLint to 5.6.1.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Update ESLint to 5.6.1.
@nodejs-github-botnodejs-github-bot added the tools Issues and PRs related to the tools directory. label Sep 28, 2018
Copy link
Member

@TrottTrott left a comment

Choose a reason for hiding this comment

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

rubber stamp

@Trott
Copy link
Member

(Lite CI sufficient since that runs the linter.)

@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 28, 2018
Copy link
Member

@jasnelljasnell left a comment

Choose a reason for hiding this comment

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

rubber stamp lgtm

Copy link
Member

@trivikrtrivikr left a comment

Choose a reason for hiding this comment

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

RSLGTM

@danbev
Copy link
Contributor

(Lite CI sufficient since that runs the linter.)

Sorry, I saw this after I started a full CI.

@danbev
Copy link
Contributor

Landed in bb00eda.

@danbevdanbev closed this Oct 1, 2018
danbev pushed a commit that referenced this pull request Oct 1, 2018
Update ESLint to 5.6.1. PR-URL: #23149 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Roman Reiss <[email protected]>
@cjihrigcjihrig deleted the eslint branch October 1, 2018 12:53
targos pushed a commit that referenced this pull request Oct 1, 2018
Update ESLint to 5.6.1. PR-URL: #23149 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Roman Reiss <[email protected]>
targos pushed a commit that referenced this pull request Oct 3, 2018
Update ESLint to 5.6.1. PR-URL: #23149 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Roman Reiss <[email protected]>
@targostargos mentioned this pull request Oct 7, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.toolsIssues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@cjihrig@nodejs-github-bot@Trott@danbev@silverwind@jasnell@trivikr