Skip to content

WRN-19252: Fix enact pack --verbose is not working - #265

Merged
hong6316 merged 1 commit into
feature/webpack5from
feature/WRN-19252
Mar 31, 2022
Merged

WRN-19252: Fix enact pack --verbose is not working#265
hong6316 merged 1 commit into
feature/webpack5from
feature/WRN-19252

Conversation

@MikyungKim

@MikyungKimMikyungKim commented Mar 29, 2022

Copy link
Copy Markdown
Contributor

In this PR, we'd like to fix enact pack --verbose is not working.
webpack's ProgressPlugin is changed to check if compiler is an instance of Compiler.
Currently, dev-utils and cli have different webpack modules so ProgressPlugin created from dev-utils cannot work correctly.
To remedy this, I've changed to create PrgressPlugin from cli's webpack module.

Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)

Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)

@hong6316hong6316 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@hong6316
hong6316 merged commit c63c563 into feature/webpack5Mar 31, 2022
@hong6316
hong6316 deleted the feature/WRN-19252 branch March 31, 2022 01:54
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 (mikyung27.kim@lge.com)
* WRN-18582: Fixed serve issue
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* WRN-18582: Added webpack cache config
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* applied more changes
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* Added changelog
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* fix
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* update `listen` to `startCallback` from webpack dev server
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* fix filename..ext issue
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* 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 <mikyung27.kim@lge.com>
* Update babel-jest version
Co-authored-by: Mikyung Kim <mikyung27.kim@lge.com>
* Update babel-plugin-dev-expression version
Co-authored-by: Mikyung Kim <mikyung27.kim@lge.com>
* remove unnecessary code
* revert file permission
* update dependencies
* update package.json
* Update package.json
Co-authored-by: Mikyung Kim <mikyung27.kim@lge.com>
* Update package.json
* fix
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
Co-authored-by: Mikyung Kim <mikyung27.kim@lge.com>
* update changelog and package.json
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
Co-authored-by: taeyoung.hong <35059065+hong6316@users.noreply.github.com>
* 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 <mikyung27.kim@lge.com>
* Update README.md
Co-authored-by: Mikyung Kim <mikyung27.kim@lge.com>
* update npm-shrinkwrap.json
* update @babel/core to the same version of dev-utils
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* Move some dependencies to dev dependencies
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
Co-authored-by: Mikyung Kim <mikyung27.kim@lge.com>
* WRN-19252: Pass ProgressPlugin to VerboseLogPlugin (#265)
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* 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 <mikyung27.kim@lge.com>
Co-authored-by: Mikyung Kim <mikyung27.kim@lge.com>
* WRN-19694: Update to React18 (#266)
* update react version
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* update testing library for react18
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* update react version
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* update dependencies
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* update version
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* update dev-utils branch
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* revert dev-utils version
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* WRN-20481: Fix screenshot test build fail (#267)
* Added node-polyfill-webpack-plugin
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* added some comments
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* link dev-utils
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* WRN-20481: Fix lint error and revert packages version
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* update change log
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
* update dependencies
Enact-DCO-1.0-Signed-off-by: Mikyung Kim (mikyung27.kim@lge.com)
Co-authored-by: taeyoung.hong <35059065+hong6316@users.noreply.github.com>
Sign up for freeto 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

@MikyungKim@hong6316