Skip to content

WRN-12124: Remove deprecated babel-eslint module #261

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

Merged
merged 17 commits into from
Mar 25, 2022

Conversation

hong6316
Copy link
Contributor

@hong6316 hong6316 commented Mar 11, 2022

Issue Resolved / Feature Added

babel-eslint has been deprecated

`babel-eslint` is now `@babel/eslint-parser`. This package will no longer receive updates.

Resolution

Removed deprecated babel-eslint

Additional Considerations

When you release the new @enact/cli version with this PR, you should update npm-shrinkwrap.json.

$ rm -rf npm-shrinkwrap.json
$ npm i
$ npm audit fix
$ npm shrinkwrap

Links

WRN-12124

Comments

Enact-DCO-1.0-Signed-off-by: Taeyoung Hong ([email protected])

@hong6316 hong6316 requested review from MikyungKim March 11, 2022 05:20
@MikyungKim MikyungKim changed the base branch from develop to feature/webpack5 March 21, 2022 07:47
Copy link
Contributor

@MikyungKim MikyungKim left a comment

Choose a reason for hiding this comment

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

babel-eslint is remained in README.md. Please update that too.

@hong6316
Copy link
Contributor Author

babel-eslint is remained in README.md. Please update that too.

@MikyungKim
I checked README.md and I remained this word first with in my opinion.
Because this description is the guide for removing previous modules that are no longer used.
(and customers will remove babel-eslint that we intended)

But anyway, I remove the word babel-eslint in README.md.

Copy link
Contributor

@MikyungKim MikyungKim left a comment

Choose a reason for hiding this comment

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

Please add a change log.

Co-authored-by: Mikyung Kim <[email protected]>
Copy link
Contributor

@MikyungKim MikyungKim left a comment

Choose a reason for hiding this comment

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

LGTM

@MikyungKim MikyungKim merged commit 826d927 into feature/webpack5 Mar 25, 2022
@MikyungKim MikyungKim deleted the feature/WRN-12124 branch March 25, 2022 05:36
hong6316 added a commit that referenced this pull request Apr 11, 2022
* WRN-18582: Migrate to webpack5 (#264)

* WRN-18582: webpack5 initial commit

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-18582: Fixed serve issue

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-18582: Added webpack cache config

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* applied more changes

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* Added changelog

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* fix

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update `listen` to `startCallback` from webpack dev server

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* fix filename..ext issue

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-16762: Added support for tailwindcss and update dependencies (#263)

* initial modules update(before wp5 update)

* npm audit fix

* Update package.json

* fix lint errors

* update CHANGELOG.md

* Update package.json

* revert CHANGELOG.mx

* revert jest 27

* revert npm-shrinkwrap.json

* revert pack/serve.js

* for removing conflict with #262, revert versions

* Update react-refresh-webpack-plugin version

Co-authored-by: Mikyung Kim <[email protected]>

* Update babel-jest version

Co-authored-by: Mikyung Kim <[email protected]>

* Update babel-plugin-dev-expression version

Co-authored-by: Mikyung Kim <[email protected]>

* remove unnecessary code

* revert file permission

* update dependencies

* update package.json

* Update package.json

Co-authored-by: Mikyung Kim <[email protected]>

* Update package.json

* fix

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

Co-authored-by: Mikyung Kim <[email protected]>

* update changelog and package.json

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

Co-authored-by: taeyoung.hong <[email protected]>

* WRN-12124: Remove deprecated babel-eslint module (#261)

* use @babel/eslint-parser

* remove babel-preset-react-app

* remove @babel/eslint-parser

* Revert "remove @babel/eslint-parser"

This reverts commit 1b8b37e.

* Revert "remove babel-preset-react-app"

This reverts commit b8579cb.

* Revert "use @babel/eslint-parser"

This reverts commit b73146a.

* babel-eslint -> @babel/eslint-parser

* unpinning enact repos

* remove babel-eslint related word in README.md

* Update README.md

* modify changelog & readme

* Update package.json

Co-authored-by: Mikyung Kim <[email protected]>

* Update README.md

Co-authored-by: Mikyung Kim <[email protected]>

* update npm-shrinkwrap.json

* update @babel/core to the same version of dev-utils

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* Move some dependencies to dev dependencies

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

Co-authored-by: Mikyung Kim <[email protected]>

* WRN-19252: Pass ProgressPlugin to VerboseLogPlugin (#265)

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-12748: Update eslint 8 related modules (#262)

* update eslint 8

* eslint 8 changes strict package exports(#14706), call another method

* revert js changes

* Update package.json

* restore eslintrc.js - 'prettier' (included 'prettier/babel', 'prettier/react')

* Update package.json

Co-authored-by: Mikyung Kim <[email protected]>

Co-authored-by: Mikyung Kim <[email protected]>

* WRN-19694: Update to React18 (#266)

* update react version

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update testing library for react18

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update react version

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update dependencies

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update version

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update dev-utils branch

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* revert dev-utils version

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-20481: Fix screenshot test build fail (#267)

* Added node-polyfill-webpack-plugin

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* added some comments

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* link dev-utils

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-20481: Fix lint error and revert packages version

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update change log

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update dependencies

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

Co-authored-by: taeyoung.hong <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants