Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

refactor!: remove the deprecated constructor API#4808

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
alexander-akait merged 2 commits intonextfromfix/rm-old-constructor-api
Apr 7, 2023

Conversation

@snitin315
Copy link
Member

@snitin315snitin315 commentedApr 7, 2023
edited
Loading

  • This is abugfix
  • This is afeature
  • This is acode refactor
  • This is atest update
  • This is adocs update
  • This is ametadata update

For Bugs and Features; did you add new tests?

Removed deprecated tests.

Motivation / Use-Case

Resolve TODO and the remove old API.

Breaking Changes

BREAKING CHANGE: The deprecated constructor API has been removed. Please useoptions as the first argument andcompiler as the second argument.

Additional Info

CLI is compatible

https://github.com/webpack/webpack-cli/blob/a67dee4facb926c3a041266632e294084542a268/packages/serve/src/index.ts#L229

@codecov
Copy link

codecovbot commentedApr 7, 2023
edited
Loading

Codecov Report

Patch coverage has no change and project coverage change:+0.10 🎉

Comparison is base(22d6f32) 92.09% compared to head(0fab758) 92.20%.

Additional details and impacted files
@@            Coverage Diff             @@##             next    #4808      +/-   ##==========================================+ Coverage   92.09%   92.20%   +0.10%==========================================  Files          15       15                Lines        1544     1539       -5       Branches      575      573       -2     ==========================================- Hits         1422     1419       -3+ Misses        113      111       -2  Partials        9        9
Impacted FilesCoverage Δ
lib/Server.js94.09% <ø> (-0.03%)⬇️

... and1 file with indirect coverage changes

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

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

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

Reviewers

@alexander-akaitalexander-akaitalexander-akait approved these changes

@hiroppyhiroppyAwaiting requested review from hiroppy

@anshumanvanshumanvAwaiting requested review from anshumanv

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@snitin315@alexander-akait

[8]ページ先頭

©2009-2025 Movatter.jp