Skip to content

tools: update ESLint to 8.4.0#41085

Merged
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
lpinca:update/eslint
Dec 6, 2021
Merged

tools: update ESLint to 8.4.0#41085
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
lpinca:update/eslint

Conversation

@lpinca

@lpinca lpinca commented Dec 4, 2021

Copy link
Copy Markdown
Member

Update ESLint to 8.4.0

@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Dec 4, 2021

@tniessen tniessen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

RSLGTM

@Trott Trott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Rubber-stamp LGTM

@lpinca lpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 6, 2021
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 6, 2021
@nodejs-github-bot nodejs-github-bot merged commit 42b51af into nodejs:master Dec 6, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 42b51af

@BridgeAR

BridgeAR commented Dec 6, 2021

Copy link
Copy Markdown
Member

Did the CI actually run for this? I guess not? This might have broken our CI. We constantly have eslint errors now:

https://ci.nodejs.org/job/node-test-commit-linux-containered/29888/#showFailuresLink

@BridgeAR BridgeAR mentioned this pull request Dec 6, 2021
@lpinca lpinca deleted the update/eslint branch December 7, 2021 05:44
danielleadams pushed a commit that referenced this pull request Dec 13, 2021
PR-URL: #41085
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
danielleadams pushed a commit that referenced this pull request Dec 14, 2021
PR-URL: #41085
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
Linkgoron pushed a commit to Linkgoron/node that referenced this pull request Jan 31, 2022
PR-URL: nodejs#41085
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
PR-URL: #41085
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
@danielleadams danielleadams mentioned this pull request Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants