Skip to content

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 18, 2024

This PR updates Next.js from version 13.5.6 to 14.2.12.

It also updates the Node.js version in the deployment workflow from v16 to v18, as this is now the minimum version required by Next.js v14 (see: https://github.com/ethereum/solidity-website/actions/runs/10924194254/job/30322639405#step:7:10).

Due to breaking changes in the Next.js configuration (see: https://nextjs.org/docs/pages/building-your-application/deploying/static-exports#version-history), I replaced the use of next export -o docs. If we still need the default output to be standalone, some refactoring will be required to support conditional output based on different environments.

Bumps [next](https://github.com/vercel/next.js) from 13.5.6 to 14.2.10.
- [Release notes](https://github.com/vercel/next.js/releases)
- [Changelog](https://github.com/vercel/next.js/blob/canary/release.js)
- [Commits](vercel/next.js@v13.5.6...v14.2.10)

---
updated-dependencies:
- dependency-name: next
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Sep 18, 2024
@ekpyron
Copy link
Contributor

ekpyron commented Sep 18, 2024

We tried applying this earlier already (see #114) and merged it (it's unfortunate that CI is not set up to run on PRs in this repo, so we had to merge to see if it works) - but it fails the builds, see https://github.com/ethereum/solidity-website/actions/runs/10924194254/job/30322639405 - likely due to also requiring a bump in the node version.
So I recreated this PR, but we can't merge it as is, but it requires more work apparently.

I haven't yet properly checked if the vulnerability conditions apply to our case and thus how urgent this is.

@ekpyron ekpyron requested a review from wackerow September 18, 2024 14:36
@ekpyron
Copy link
Contributor

ekpyron commented Sep 18, 2024

@wackerow is this something you can help with / take care of or should we look into this?

@@ -1,7 +1,8 @@
/** @type {import('next').NextConfig} */
const nextConfig = {
reactStrictMode: true,
output: 'standalone',
Copy link
Member

@r0qs r0qs Sep 19, 2024

Choose a reason for hiding this comment

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

Why was this standalone by default? Should we still keep such behavior? I guess we can always set it to export for the website, no?

@r0qs r0qs changed the title Bump next from 13.5.6 to 14.2.10 Bump next from 13.5.6 to 14.2.12 Sep 19, 2024
@r0qs
Copy link
Member

r0qs commented Feb 19, 2025

Hey @wackerow could you take a look at this PR when you have some time? Mainly about this question here: #116 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants