Skip to content

Replace babel-eslint with @babel/eslint-parser#267

Open
henhal wants to merge 2 commits into
AnomalyInnovations:masterfrom
henhal:babel-eslint
Open

Replace babel-eslint with @babel/eslint-parser#267
henhal wants to merge 2 commits into
AnomalyInnovations:masterfrom
henhal:babel-eslint

Conversation

@henhal

@henhal henhal commented Oct 20, 2021

Copy link
Copy Markdown

As per output during install:

warning serverless-bundle > babel-eslint@10.1.0: babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.

@jayair jayair added the internal An internal tooling related change label Oct 20, 2021
@jayair

jayair commented Oct 20, 2021

Copy link
Copy Markdown
Member

Hmm not sure what's causing the tests to fail here.

@samatcd

samatcd commented Nov 25, 2021

Copy link
Copy Markdown

@jayair my guess is that "parser": "babel-eslint" config in serverless-bundle/src/eslintrc.json needs to change to @babel/eslint-parser.

@jayair

jayair commented Nov 25, 2021

Copy link
Copy Markdown
Member

Oh I see. Anybody want to take this on and fix the tests so we can merge it?

@Ionut-Milas

Copy link
Copy Markdown

Any update on this ?

@jayair

jayair commented Jan 17, 2022

Copy link
Copy Markdown
Member

If somebody could figure out what's going on with the tests, then I can merge this and publish it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal An internal tooling related change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants