Skip to content
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

chore: update dependencies to the latest version #1038

Closed

Conversation

ersachin3112
Copy link
Contributor

@ersachin3112 ersachin3112 commented Jun 5, 2023

This PR contains a:

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

Motivation / Use-Case

update dependencies to the latest version

Breaking Changes

No

Additional Info

No

@alexander-akait
Copy link
Member

Need fix tests

@codecov
Copy link

codecov bot commented Jun 11, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.24 ⚠️

Comparison is base (f2cea2f) 90.54% compared to head (3c11027) 90.31%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1038      +/-   ##
==========================================
- Coverage   90.54%   90.31%   -0.24%     
==========================================
  Files           5        5              
  Lines         857      857              
  Branches      243      244       +1     
==========================================
- Hits          776      774       -2     
- Misses         71       73       +2     
  Partials       10       10              
Impacted Files Coverage Δ
src/index.js 96.13% <100.00%> (ø)
src/loader.js 89.06% <100.00%> (-1.05%) ⬇️

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

src/loader.js Outdated
@@ -349,6 +349,7 @@ function pitch(request) {
const outputOptions = {
filename: childFilename,
publicPath,
asyncChunkLoading: false,
Copy link
Member

Choose a reason for hiding this comment

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

We should not do it or fix it on webpack side

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

@ersachin3112 I think no

new NodeTemplatePlugin(outputOptions).apply(childCompiler);
new NodeTemplatePlugin().apply(childCompiler);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@alexander-akait We have only filename and publicPath in outputOptions but the NodeTemplatePlugin only uses asyncChunkLoading. I think don't need to pass it here, TS also warns about the same.

Screenshot 2023-06-15 at 6 14 48 AM

@alexander-akait
Copy link
Member

Let's fix old API

@ersachin3112 ersachin3112 deleted the deps-up-date branch September 5, 2023 00:41
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.

None yet

2 participants