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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[examples] Update next.js boilerplate example #9008

Closed
wants to merge 2 commits into from

Conversation

iteratetograceness
Copy link
Contributor

@iteratetograceness iteratetograceness commented Dec 5, 2022

Updates the Next.js Boilerplate example with the create-next-app redesign.

Related Issues

Related to vercel/next.js#43482.

馃搵 Checklist

Tests

  • The code changed/added as part of this PR has been covered with tests
  • All tests pass locally with yarn test-unit

Code Review

  • This PR has a concise title and thorough description useful to a reviewer
  • Issue from task tracker has a link to this PR

@iteratetograceness iteratetograceness self-assigned this Dec 5, 2022
@@ -1,6 +1,7 @@
/** @type {import('next').NextConfig} */
const nextConfig = {
reactStrictMode: true,
swcMinify: true,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
swcMinify: true,

This is the default now so we can omit it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome will do + I'll make a PR to omit this from the create-next-app package templates as well! Also in regards to merging this PR, I think it would be best to hold off until v13.0.7 is officially released -- lmk what you think!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ijjk re: @styfle's comment below: ah, looks like this template will update itself once 13.0.7 becomes an official release -- so I'm assuming this PR is not needed!

But I'll still open the PR to omit swcMinify from the create-next-app templates!

cc: @kale-stew

Copy link
Member

@styfle styfle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this updated in create-next-app?

We already have a script that can automatically update this example (see #8998 for example)

@styfle styfle deleted the update-nextjs-example branch December 5, 2022 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: examples semver: minor PR contains new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants