-
Notifications
You must be signed in to change notification settings - Fork 26k
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: change next build
to emit output with output: export
#47376
Merged
timneutkens
merged 15 commits into
canary
from
styfle/next-868-change-next-build-to-emit-out
Mar 23, 2023
Merged
Changes from 14 commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
1531103
feat: change `next build` to emit output with `output: export`
styfle f842440
Merge branch 'canary' into styfle/next-868-change-next-build-to-emit-out
styfle 6d03629
Fix merge conflict
styfle fb64c22
Add tests for `next export` no-op
styfle 6dc82a3
Allow `distDir` to configure the `outDir`
styfle 436b8cd
Document the usage of `distDir: out` with `output: export`
styfle 1d11a13
Merge branch 'canary' into styfle/next-868-change-next-build-to-emit-out
styfle 0b59bf2
Merge branch 'canary' into styfle/next-868-change-next-build-to-emit-out
styfle b5b9565
Update docs
styfle 5f52670
Add deprecation message
styfle 4f705d4
Minor refactor to tests to hoist expected files up
styfle 10da183
Merge branch 'canary' into styfle/next-868-change-next-build-to-emit-out
styfle 0345b26
Mention trailing slash usage
styfle ec62cdf
Merge branch 'canary' into styfle/next-868-change-next-build-to-emit-out
styfle 59e315f
Merge branch 'canary' into styfle/next-868-change-next-build-to-emit-out
kodiakhq[bot] File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 think this might have been a bug before.
Previously, it only read
nextConfig
if it was passed in as an arg, but not the case when it read the file on line 181 above