Skip to content

feat: allow passing strings in env flag - #1939

Merged
alexander-akait merged 4 commits into
webpack:masterfrom
anshumanv:enh-env
Oct 16, 2020
Merged

alexander-akait merged 4 commits into
webpack:masterfrom
anshumanv:enh-env

Conversation

@anshumanv

Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?
feat/fix

Did you add tests for your changes?
Yes

If relevant, did you update the documentation?
Need to reflect in docs

Summary

  • Allow passing strings to env flag which is passed to config
  • You can now do -

webpack --env platform=app --env production

which will pass { platform: "app", production: true } to the config

Does this PR introduce a breaking change?

Nope

Other information

Fixes #1932

@anshumanv
anshumanv requested a review from a team as a code owner October 13, 2020 17:43
@paulmackinlay

Copy link
Copy Markdown

@anshumanv

Copy link
Copy Markdown
Contributor Author

Yep, @paulmackinlay I have a PR already - webpack/webpack.js.org#4051 hoping it's merged soon.

@alexander-akait

Copy link
Copy Markdown
Member

/cc @anshumanv please focus on it

@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.

Can we add tests?

marques-work added a commit to gocd/gocd that referenced this pull request Oct 15, 2020
@alexander-akait
alexander-akait merged commit cc081a2 into webpack:master Oct 16, 2020
@anshumanv
anshumanv deleted the enh-env branch October 16, 2020 14:10
@ntucker

ntucker commented Oct 16, 2020

Copy link
Copy Markdown

Release new version? :D

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.

Enhance env flag support

6 participants