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 babel-eslint-parser to 7.16.0 #40720

Closed
wants to merge 2 commits into from
Closed

Conversation

Trott
Copy link
Member

@Trott Trott commented Nov 4, 2021

No description provided.

@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Nov 4, 2021
@targos
Copy link
Member

targos commented Nov 4, 2021

Looks like some files have to be fixed.

@targos
Copy link
Member

targos commented Nov 4, 2021

s/will be flagged when we flagged when we/will be flagged when we/

@Trott Trott added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Nov 4, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 6, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2021

Commit Queue failed
- Loading data for nodejs/node/pull/40720
✔  Done loading data for nodejs/node/pull/40720
----------------------------------- PR info ------------------------------------
Title      tools: update babel-eslint-parser to 7.16.0 (#40720)
Author     Rich Trott  (@Trott)
Branch     Trott:bep-7.16 -> nodejs:master
Labels     tools, author ready
Commits    2
 - test: add semicolons for linter update
 - tools: update babel-eslint-parser to 7.16.0
Committers 1
 - Rich Trott 
PR-URL: https://github.com/nodejs/node/pull/40720
Reviewed-By: Tobias Nießen 
Reviewed-By: Michaël Zasso 
Reviewed-By: Luigi Pinca 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40720
Reviewed-By: Tobias Nießen 
Reviewed-By: Michaël Zasso 
Reviewed-By: Luigi Pinca 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 04 Nov 2021 05:41:28 GMT
   ✔  Approvals: 3
   ✔  - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/40720#pullrequestreview-797837612
   ✔  - Michaël Zasso (@targos) (TSC): https://github.com/nodejs/node/pull/40720#pullrequestreview-797840350
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/40720#pullrequestreview-798182790
   ✔  Last GitHub Actions successful
   ✖  No Jenkins CI runs detected
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1429133588

@github-actions github-actions bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Nov 6, 2021
@targos targos added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 6, 2021
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 6, 2021
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@Trott Trott removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 8, 2021
Add missing semicolons that will be flagged when we update
@babel/eslint-parser to 7.16.0.

PR-URL: nodejs#40720
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: nodejs#40720
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@Trott
Copy link
Member Author

Trott commented Nov 8, 2021

Landed in 04451d8...9ef0ae2

@Trott Trott closed this Nov 8, 2021
@Trott Trott deleted the bep-7.16 branch November 8, 2021 21:17
targos pushed a commit that referenced this pull request Nov 21, 2021
Add missing semicolons that will be flagged when we update
@babel/eslint-parser to 7.16.0.

PR-URL: #40720
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this pull request Nov 21, 2021
PR-URL: #40720
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@danielleadams
Copy link
Contributor

danielleadams commented Jan 30, 2022

@Trott do you mind backporting this to 16.x? When I tried to land in 16.x-staging, there was a test that didn't pass.

Trott added a commit to Trott/io.js that referenced this pull request Jan 30, 2022
Add missing semicolons that will be flagged when we update
@babel/eslint-parser to 7.16.0.

PR-URL: nodejs#40720
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Trott added a commit to Trott/io.js that referenced this pull request Jan 30, 2022
PR-URL: nodejs#40720
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@Trott
Copy link
Member Author

Trott commented Jan 30, 2022

@Trott do you mind backporting this to 16.x? When I tried to land in 16.x-staging, there was a test that didn't pass.

@danielleadams Backported in #41770

Trott added a commit to Trott/io.js that referenced this pull request Jan 31, 2022
Add missing semicolons that will be flagged when we update
@babel/eslint-parser to 7.16.0.

PR-URL: nodejs#40720
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Trott added a commit to Trott/io.js that referenced this pull request Jan 31, 2022
PR-URL: nodejs#40720
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
Add missing semicolons that will be flagged when we update
@babel/eslint-parser to 7.16.0.

PR-URL: #40720
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>

Backport-PR-URL: #41770
Reviewed-By: Danielle Adams <[email protected]>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
PR-URL: #40720
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>

Backport-PR-URL: #41770
Reviewed-By: Danielle Adams <[email protected]>
@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.

6 participants