Skip to content

feat!: minimum supported webpack version is 5.0.0 #1389

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 2 commits into from
Nov 17, 2022

Conversation

snitin315
Copy link
Member

@snitin315 snitin315 commented Nov 17, 2022

This PR contains a:

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

Motivation / Use-Case

Drop webpack v4

Breaking Changes

minimum supported webpack version is 5.0.0

Additional Info

@snitin315 snitin315 changed the title feat!: minimium supported webpack version is 5.0.0 feat!: minimum supported webpack version is 5.0.0 Nov 17, 2022
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.

Looks like we remove more than expected? Can you check?

Comment on lines -20 to -33
// Todo remove when we drop webpack@4 support
if (typeof outputFileSystemFromOptions.join !== "function") {
throw new Error(
"Invalid options: options.outputFileSystem.join() method is expected"
);
}

// Todo remove when we drop webpack@4 support
// @ts-ignore
if (typeof outputFileSystemFromOptions.mkdirp !== "function") {
throw new Error(
"Invalid options: options.outputFileSystem.mkdirp() method is expected"
);
}
Copy link
Member Author

Choose a reason for hiding this comment

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

@alexander-akait Do we need this with webpack v5?

Copy link
Member Author

Choose a reason for hiding this comment

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

Otherwise, I'll remove test cases for this as well, which are currently failing.

Copy link
Member

Choose a reason for hiding this comment

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

If this tests for webpack - Yes

Copy link
Member

Choose a reason for hiding this comment

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

We have a lot of integration tests, so we will catch if it will have problems

@codecov
Copy link

codecov bot commented Nov 17, 2022

Codecov Report

Base: 97.48% // Head: 97.34% // Decreases project coverage by -0.13% ⚠️

Coverage data is based on head (f929794) compared to base (02a2b33).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1389      +/-   ##
==========================================
- Coverage   97.48%   97.34%   -0.14%     
==========================================
  Files           9        9              
  Lines         398      377      -21     
  Branches      123      112      -11     
==========================================
- Hits          388      367      -21     
  Misses          9        9              
  Partials        1        1              
Impacted Files Coverage Δ
src/utils/setupHooks.js 94.64% <ø> (-1.14%) ⬇️
src/utils/setupOutputFileSystem.js 100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@alexander-akait alexander-akait merged commit 88bc300 into master Nov 17, 2022
@alexander-akait alexander-akait deleted the drop-webpack-4 branch November 17, 2022 01:47
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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy