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

[docs-infra] Upgrade next to v14.2.14 #43964

Closed
wants to merge 3 commits into from

Conversation

aarongarciah
Copy link
Member

@aarongarciah aarongarciah commented Oct 2, 2024

Next.js v14.2.14 introduced a change that broke our docs build. We discovered this after running pnpm dedupe in a release PR. See #43957 (comment) for more info.

This change in Next.js was the culprit: https://github.com/vercel/next.js/pull/70382/files#diff-5f441142eb8ba9bcba515aa36fbc420a113307bd191e54a4e9e6c43529fc2613R861

@aarongarciah aarongarciah added dependencies Update of dependencies nextjs labels Oct 2, 2024
@aarongarciah aarongarciah requested a review from Janpot October 2, 2024 09:07
@@ -56,7 +56,8 @@ export default withDocsInfra({
// We only care about Node runtime at this point.
(options.nextRuntime === undefined || options.nextRuntime === 'nodejs')
) {
const [nextExternals, ...externals] = config.externals;
const externals = config.externals.slice(0, config.externals.length - 2);
const nextExternals = config.externals.at(-1);
Copy link
Member Author

Choose a reason for hiding this comment

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

nextExternals is the last element in the array since Next.js v14.2.14

@aarongarciah aarongarciah added the scope: docs-infra Specific to the docs-infra product label Oct 2, 2024
@aarongarciah aarongarciah changed the title [core] Upgrade next to v14.2.14 [docs-infra] Upgrade next to v14.2.14 Oct 2, 2024
docs/next.config.mjs Outdated Show resolved Hide resolved
@aarongarciah aarongarciah mentioned this pull request Oct 2, 2024
@mui-bot
Copy link

mui-bot commented Oct 2, 2024

Netlify deploy preview

https://deploy-preview-43964--material-ui.netlify.app/

Bundle size report

Bundle size will be reported once CircleCI build #755879 finishes.

Generated by 🚫 dangerJS against 6c75573

@aarongarciah
Copy link
Member Author

Ouch, more problems after pnpm dedupe; this time some types from @mui/base: https://github.com/mui/material-ui/actions/runs/11141069280/job/30961190258?pr=43964#step:7:443

@aarongarciah
Copy link
Member Author

This is fix is now part of #43555

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Update of dependencies nextjs scope: docs-infra Specific to the docs-infra product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants