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

tools: update eslint to 8.36.0 #47046

Merged
merged 2 commits into from Mar 14, 2023
Merged

Conversation

nodejs-github-bot
Copy link
Collaborator

This is an automated update of eslint to 8.36.0.

@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Mar 12, 2023
@MoLow MoLow added the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 12, 2023
@nodejs-github-bot nodejs-github-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 Mar 14, 2023
@nodejs-github-bot
Copy link
Collaborator Author

Commit Queue failed
- Loading data for nodejs/node/pull/47046
✔  Done loading data for nodejs/node/pull/47046
----------------------------------- PR info ------------------------------------
Title      tools: update eslint to 8.36.0 (#47046)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     nodejs-github-bot:actions/tools-update-eslint -> nodejs:main
Labels     tools
Commits    2
 - tools: update eslint to 8.36.0
 - tools: update eslint to 8.36.0
Committers 1
 - Node.js GitHub Bot 
PR-URL: https://github.com/nodejs/node/pull/47046
Reviewed-By: Rich Trott 
Reviewed-By: Moshe Atlow 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Luigi Pinca 
Reviewed-By: Colin Ihrig 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/47046
Reviewed-By: Rich Trott 
Reviewed-By: Moshe Atlow 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Luigi Pinca 
Reviewed-By: Colin Ihrig 
--------------------------------------------------------------------------------
   ⚠  Commits were pushed since the last review:
   ⚠  - tools: update eslint to 8.36.0
   ℹ  This PR was created on Sun, 12 Mar 2023 00:28:24 GMT
   ✔  Approvals: 5
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/47046#pullrequestreview-1335948613
   ✔  - Moshe Atlow (@MoLow): https://github.com/nodejs/node/pull/47046#pullrequestreview-1335963574
   ✔  - Yagiz Nizipli (@anonrig): https://github.com/nodejs/node/pull/47046#pullrequestreview-1335966215
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/47046#pullrequestreview-1335967651
   ✔  - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/47046#pullrequestreview-1336182103
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/4410989878

@MoLow MoLow added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Mar 14, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 14, 2023
@nodejs-github-bot
Copy link
Collaborator Author

Commit Queue failed
- Loading data for nodejs/node/pull/47046
✔  Done loading data for nodejs/node/pull/47046
----------------------------------- PR info ------------------------------------
Title      tools: update eslint to 8.36.0 (#47046)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     nodejs-github-bot:actions/tools-update-eslint -> nodejs:main
Labels     tools
Commits    2
 - tools: update eslint to 8.36.0
 - tools: update eslint to 8.36.0
Committers 1
 - Node.js GitHub Bot 
PR-URL: https://github.com/nodejs/node/pull/47046
Reviewed-By: Rich Trott 
Reviewed-By: Moshe Atlow 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Luigi Pinca 
Reviewed-By: Colin Ihrig 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/47046
Reviewed-By: Rich Trott 
Reviewed-By: Moshe Atlow 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Luigi Pinca 
Reviewed-By: Colin Ihrig 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sun, 12 Mar 2023 00:28:24 GMT
   ✔  Approvals: 5
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/47046#pullrequestreview-1335948613
   ✔  - Moshe Atlow (@MoLow): https://github.com/nodejs/node/pull/47046#pullrequestreview-1338718407
   ✔  - Yagiz Nizipli (@anonrig): https://github.com/nodejs/node/pull/47046#pullrequestreview-1335966215
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/47046#pullrequestreview-1335967651
   ✔  - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/47046#pullrequestreview-1336182103
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  No git cherry-pick in progress
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
 * branch                  main       -> FETCH_HEAD
✔  origin/main is now up-to-date
- Downloading patch for 47046
From https://github.com/nodejs/node
 * branch                  refs/pull/47046/merge -> FETCH_HEAD
✔  Fetched commits as f41d50786286..1da9ecd32324
--------------------------------------------------------------------------------
[main fd03fdbbbf] tools: update eslint to 8.36.0
 Date: Sun Mar 12 00:28:20 2023 +0000
 863 files changed, 1906 insertions(+), 2448 deletions(-)
 rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/eslint-utils/LICENSE (100%)
 rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/eslint-utils/index.js (98%)
 rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/eslint-utils/index.mjs (98%)
 create mode 100644 tools/node_modules/eslint/node_modules/@eslint-community/eslint-utils/package.json
 rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/regexpp/LICENSE (100%)
 rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/regexpp/index.js (77%)
 rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/regexpp/index.mjs (77%)
 rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/regexpp/package.json (56%)
 create mode 100644 tools/node_modules/eslint/node_modules/caniuse-lite/data/features/css-text-box-trim.js
 delete mode 100644 tools/node_modules/eslint/node_modules/eslint-utils/node_modules/eslint-visitor-keys/LICENSE
 delete mode 100644 tools/node_modules/eslint/node_modules/eslint-utils/node_modules/eslint-visitor-keys/lib/index.js
 delete mode 100644 tools/node_modules/eslint/node_modules/eslint-utils/node_modules/eslint-visitor-keys/lib/visitor-keys.json
 delete mode 100644 tools/node_modules/eslint/node_modules/eslint-utils/node_modules/eslint-visitor-keys/package.json
 delete mode 100644 tools/node_modules/eslint/node_modules/eslint-utils/package.json
[main 6af79e64da] tools: update eslint to 8.36.0
 Date: Mon Mar 13 11:30:25 2023 +0000
 4 files changed, 4 insertions(+), 4 deletions(-)
   ✔  Patches applied
There are 2 commits in the PR. Attempting autorebase.
Rebasing (2/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
tools: update eslint to 8.36.0

PR-URL: #47046
Reviewed-By: Rich Trott rtrott@gmail.com
Reviewed-By: Moshe Atlow moshe@atlow.co.il
Reviewed-By: Yagiz Nizipli yagiz@nizipli.com
Reviewed-By: Luigi Pinca luigipinca@gmail.com
Reviewed-By: Colin Ihrig cjihrig@gmail.com

[detached HEAD 70e31fcd88] tools: update eslint to 8.36.0
Date: Sun Mar 12 00:28:20 2023 +0000
863 files changed, 1906 insertions(+), 2448 deletions(-)
rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/eslint-utils/LICENSE (100%)
rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/eslint-utils/index.js (98%)
rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/eslint-utils/index.mjs (98%)
create mode 100644 tools/node_modules/eslint/node_modules/@eslint-community/eslint-utils/package.json
rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/regexpp/LICENSE (100%)
rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/regexpp/index.js (77%)
rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/regexpp/index.mjs (77%)
rename tools/node_modules/eslint/node_modules/{ => @eslint-community}/regexpp/package.json (56%)
create mode 100644 tools/node_modules/eslint/node_modules/caniuse-lite/data/features/css-text-box-trim.js
delete mode 100644 tools/node_modules/eslint/node_modules/eslint-utils/node_modules/eslint-visitor-keys/LICENSE
delete mode 100644 tools/node_modules/eslint/node_modules/eslint-utils/node_modules/eslint-visitor-keys/lib/index.js
delete mode 100644 tools/node_modules/eslint/node_modules/eslint-utils/node_modules/eslint-visitor-keys/lib/visitor-keys.json
delete mode 100644 tools/node_modules/eslint/node_modules/eslint-utils/node_modules/eslint-visitor-keys/package.json
delete mode 100644 tools/node_modules/eslint/node_modules/eslint-utils/package.json
Rebasing (3/4)
Rebasing (4/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
tools: update eslint to 8.36.0

PR-URL: #47046
Reviewed-By: Rich Trott rtrott@gmail.com
Reviewed-By: Moshe Atlow moshe@atlow.co.il
Reviewed-By: Yagiz Nizipli yagiz@nizipli.com
Reviewed-By: Luigi Pinca luigipinca@gmail.com
Reviewed-By: Colin Ihrig cjihrig@gmail.com

[detached HEAD ba7ab71210] tools: update eslint to 8.36.0
Date: Mon Mar 13 11:30:25 2023 +0000
4 files changed, 4 insertions(+), 4 deletions(-)

Successfully rebased and updated refs/heads/main.

ℹ Add commit-queue-squash label to land the PR as one commit, or commit-queue-rebase to land as separate commits.

https://github.com/nodejs/node/actions/runs/4413777050

@nodejs-github-bot nodejs-github-bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Mar 14, 2023
@MoLow MoLow added commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Mar 14, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 14, 2023
@nodejs-github-bot nodejs-github-bot merged commit aa93c8b into main Mar 14, 2023
10 checks passed
@nodejs-github-bot
Copy link
Collaborator Author

Landed in aa93c8b

@nodejs-github-bot nodejs-github-bot deleted the actions/tools-update-eslint branch March 14, 2023 08:51
targos pushed a commit that referenced this pull request Mar 18, 2023
PR-URL: #47046
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
danielleadams pushed a commit that referenced this pull request Jul 6, 2023
PR-URL: #47046
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants