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.

Sign up for GitHub

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

Jump to bottom

Convert Next config file to ECMAScript module #624

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Convert Next config file to ECMAScript module #624

wants to merge 1 commit into from

Conversation

SebiVPS
Copy link
Contributor

@SebiVPS SebiVPS commented Dec 19, 2024 •
edited
Loading

Description

https://nextjs.org/docs/pages/api-reference/config/next-config-js#ecmascript-modules

When we want to use ECMA script modules (used by panda-css or next-yak for eg.) we need to update the next-config file from js to mjs.

@SebiVPS SebiVPS self-assigned this Dec 19, 2024
@nsams
Copy link
Member

nsams commented Dec 19, 2024

in next 15 we can use .ts

@johnnyomair johnnyomair changed the title update next config: use mjs instead of js Convert Next config file to ECMAScript module Dec 19, 2024
@SebiVPS SebiVPS removed the request for review from mariokemetinger January 7, 2025 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@johnnyomair johnnyomair johnnyomair approved these changes

@thomasdax98 thomasdax98 thomasdax98 approved these changes

@dkarnutsch dkarnutsch dkarnutsch approved these changes

@fraxachun fraxachun fraxachun approved these changes

@nsams nsams nsams approved these changes

@manuelblum manuelblum Awaiting requested review from manuelblum

Assignees

@SebiVPS SebiVPS

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@SebiVPS @nsams @fraxachun @dkarnutsch @thomasdax98 @johnnyomair