-
Notifications
You must be signed in to change notification settings - Fork 184
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
Add cli option for webpack-config #709
Add cli option for webpack-config #709
Conversation
@@ -112,7 +112,7 @@ export default (webpackConfig) => { | |||
} | |||
``` | |||
|
|||
In the `.reactserverrc` file add an option for `webpack-config` that points to that function file and when React Server is setting up Webpack it will call your function with the result of the built in Webpack options, allowing you to make any modifications needed. | |||
In the `.reactserverrc` file add an option for `webpackConfig` that points to that function file and when React Server is setting up Webpack it will call your function with the result of the built in Webpack options, allowing you to make any modifications needed. On the command line use the option `--webpack-config` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gigabo i am the worst at documentation so let me know what you think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I might use wording that makes it clear that this is an alternative to the option in the config file. As it stands it's possible to misinterpret this to mean that even with the option in the config file you must also pass a CLI option to trigger its use.
Maybe something like:
This may also be specified on the command line with the
--webpack-config=<FILE>
option.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, thanks @sresant! 👍
@sresant Please rebase this when you get a chance. |
d22af2b
to
cd05c7e
Compare
Currently do not have a command line option to pass a path to our webpack-config. This creates that option.