Skip to content
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

feat(nextjs): allow custom distDir #4885

Merged
merged 1 commit into from
Feb 26, 2021

Conversation

mandarini
Copy link
Member

@mandarini mandarini commented Feb 24, 2021

Current Behavior

Nextjs builder does not take into accound custom distDir that users set in their next.config.js file

Expected Behavior

Nextjs builder takes into accound custom distDir that users set in their next.config.js file

Related Issue(s)

Fixes #

@mandarini
Copy link
Member Author

mandarini commented Feb 24, 2021

@myspivey Hey James! I tried this fix, which will allow a user to set a custom distDir. Can you try it out and let me know if it does what it's expected to be doing?

@mandarini mandarini changed the title feat(nextjs): allow custom distDir and outdir Draft: feat(nextjs): allow custom distDir and outdir Feb 24, 2021
@nx-cloud
Copy link

nx-cloud bot commented Feb 24, 2021

Nx Cloud Report

CI ran the following commands for commit b5f3582. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

Status Command
#000000 nx run-many --target=build --all --parallel
#000000 nx run-many --target=e2e --projects=e2e-next
#000000 nx run-many --target=lint --all --parallel
#000000 nx run-many --target=test --all --parallel

Sent with 💌 from NxCloud.

@mandarini mandarini force-pushed the feat/nextjs-accept-custom-distDir branch from 5244304 to e6d8902 Compare February 24, 2021 17:12
@mandarini mandarini changed the title Draft: feat(nextjs): allow custom distDir and outdir Draft: feat(nextjs): allow custom distDir Feb 24, 2021
@mandarini mandarini changed the title Draft: feat(nextjs): allow custom distDir feat(nextjs): allow custom distDir Feb 24, 2021
@mandarini mandarini force-pushed the feat/nextjs-accept-custom-distDir branch from e6d8902 to dfcaf20 Compare February 24, 2021 18:02
@mandarini mandarini force-pushed the feat/nextjs-accept-custom-distDir branch from dfcaf20 to b5f3582 Compare February 24, 2021 18:29
@mandarini mandarini added the scope: nextjs Issues related to NextJS support for Nx label Feb 24, 2021
@mandarini mandarini self-assigned this Feb 24, 2021
@github-actions
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
scope: nextjs Issues related to NextJS support for Nx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants