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
fix(web): fix handling of buildLibsFromSource: false in dev-server executor #9326
fix(web): fix handling of buildLibsFromSource: false in dev-server executor #9326
Conversation
…ecutor * modify buildOptions.tsConfig before creating Webpack config * do not join buildOptions.tsConfig to root path as it is already an absolute path
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/nrwl/nx-dev/HpoccT91KvU1EpQPruRf6XkHUawG |
buildLibsFromSource: false
in dev-server
executorThere 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.
Thank you for your contribution! LGTM 🎉
…ecutor (#9326) * modify buildOptions.tsConfig before creating Webpack config * do not join buildOptions.tsConfig to root path as it is already an absolute path
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
fix(web): fix handling of
buildLibsFromSource: false
indev-server
executorbuildOptions.tsConfig
before creating Webpack configbuildOptions.tsConfig
to root path as it is already an absolute pathCurrent Behavior
The
dev-server
executor of@nrwl/web
doesn't respect the optionbuildLibsFromSource: false
. There are two causes of this behavior:buildOptions.tsConfig
needed forbuildLibsFromSource: false
is applied after creating the Webpack config while it should be done before creating the Webpack config.buildOptions.tsConfig
is joined tocontext.root
although it is already an absolute path.Note that the
webpack
executor, which has the same option, already works correctly.Expected Behavior
The
dev-server
executor of@nrwl/web
respects the optionbuildLibsFromSource: false
. If this option is set, existing build artifacts of libraries are used instead of rebuilding them from source.