Skip to content

Conversation

@vankop
Copy link
Member

@vankop vankop commented Feb 22, 2022

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

webpack/webpack#15172

Breaking Changes

no

Additional Info

need webpack/webpack#15430 and release webpack

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

I will add more test to verify #872 is fixed

@codecov
Copy link

codecov bot commented Mar 3, 2022

Codecov Report

Merging #915 (febbb7e) into master (b2261c4) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #915 +/- ## ========================================== + Coverage 90.29% 90.33% +0.04%  ========================================== Files 5 5 Lines 824 828 +4 Branches 220 221 +1 ========================================== + Hits 744 748 +4  Misses 70 70 Partials 10 10 
Impacted Files Coverage Δ
src/loader.js 88.95% <ø> (ø)
src/index.js 96.30% <100.00%> (+0.02%) ⬆️
src/utils.js 57.83% <100.00%> (+0.51%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b2261c4...febbb7e. Read the comment docs.

@vankop vankop force-pushed the fix/webpack-15172 branch from 46d24f3 to ba299b1 Compare March 3, 2022 17:27
@vankop vankop force-pushed the fix/webpack-15172 branch from ba299b1 to febbb7e Compare March 3, 2022 17:37
@alexander-akait alexander-akait merged commit 6004d95 into master Mar 3, 2022
@alexander-akait alexander-akait deleted the fix/webpack-15172 branch March 3, 2022 20:10
@rodoabad
Copy link

rodoabad commented Mar 5, 2022

Screen Shot 2022-03-04 at 10 51 13 PM

Was trying out the new version and was getting this error afterwards with experiments.outputModule: true and output.module: true. I have publicPath: 'auto'.

@vankop
Copy link
Member Author

vankop commented Mar 5, 2022

@rodoabad did you use latest webpack? webpack@5.70

@rodoabad
Copy link

rodoabad commented Mar 9, 2022

Yep I did.

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

Labels

None yet

4 participants