Update next export documentation.
#31465
Merged
Conversation
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
leerob
requested review from
huozhi,
ijjk,
lfades,
molebox,
padmaia,
shuding,
styfle and
timneutkens
as code owners
ijjk
added
created-by: Next.js team
mcsdevv
previously requested changes
Nov 15, 2021
molebox
requested changes
Nov 15, 2021
timneutkens
reviewed
Nov 16, 2021
Co-authored-by: Rich Haines <hello@richardhaines.dev> Co-authored-by: Matthew Sweeney <mail@mcs.dev>
leerob
commented
Nov 16, 2021
leerob
commented
Nov 28, 2021
|
Sorry for the slow update, this is now ready for review again 😄 |
|
@leerob @MaedahBatool looks like we have soem cross over with 2 PRs here: #31906 |
molebox
previously approved these changes
Nov 29, 2021
There was a problem hiding this comment.
Left one comment regarding a more descriptive heading. Other than that LGTM. There is another PR open that is also working on this section: #31906. Might be good to reach out to @MaedahBatool
leerob
commented
Nov 30, 2021
|
@molebox Fixed! Ready to ship 😄 |
3 tasks
2 tasks
kodiakhq bot
pushed a commit
that referenced
this pull request
Jan 5, 2022
Building off #31465 for `next export` docs. - Clearly explains the standard output from `next build` - Move Dockerfile snippet to example, instead of embedded directly - Clarify both Vercel and `next start` use the Build API output spec - Less emphasis on recommending Vercel (more neutrality) - Mention Middleware & Edge Functions when discussing Vercel - Add "Going to Production" link at the bottom for related reading
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
This more clearly lists what features are supported and not supported by
next export.