Skip to content

refactor: 💡 remove defaults flag#1543

Merged
alexander-akait merged 3 commits into
webpack:nextfrom
anshumanv:rm/defaults
May 15, 2020
Merged

refactor: 💡 remove defaults flag#1543
alexander-akait merged 3 commits into
webpack:nextfrom
anshumanv:rm/defaults

Conversation

@anshumanv

@anshumanv anshumanv commented May 13, 2020

Copy link
Copy Markdown
Contributor

drop support for defaults flag

What kind of change does this PR introduce?

Drop support for defaults flag.

Did you add tests for your changes?
Removed corresponding tests

If relevant, did you update the documentation?
Yes

Summary

Does this PR introduce a breaking change?
Shouldn't break anything.

Other information

drop support for defaults flag
@anshumanv
anshumanv marked this pull request as ready for review May 13, 2020 18:18
@anshumanv
anshumanv requested a review from a team as a code owner May 13, 2020 18:18

@alexander-akait alexander-akait left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, good job!

@alexander-akait

Copy link
Copy Markdown
Member

/cc @webpack/cli-team

@snitin315

Copy link
Copy Markdown
Member

One more place to remove => docs

@anshumanv

anshumanv commented May 14, 2020

Copy link
Copy Markdown
Contributor Author

Removed 👍, good find!

@anshumanv anshumanv closed this May 15, 2020
@anshumanv anshumanv reopened this May 15, 2020
@anshumanv
anshumanv requested a review from alexander-akait May 15, 2020 05:19
@alexander-akait
alexander-akait merged commit a905590 into webpack:next May 15, 2020
@alexander-akait

Copy link
Copy Markdown
Member

Good job! Thanks!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants