-
Notifications
You must be signed in to change notification settings - Fork 27.6k
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
Enable typechecking on config #57892
Conversation
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
@balazsorban44 thanks for the approval. I guess CI is currently broken in |
The merging process is automatic. It might take a bit but will be merged. |
ah nice, fancy. I hope I didn't break that automatic process by hitting the "Update branch" button to merge from |
@balazsorban44 it seems this still hasn't been automatically merged. Should I do something? Or can you, from your side? |
@balazsorban44 Seems like the only thing that is failing now is the Datadog failure, seemingly unrelated to this PR:
What do you think of ignoring the CI error and merging as is? |
Thanks for the review and merge @balazsorban44 @huozhi 🙌 |
What?
Enable TS type-checking on config for reproduction templates
Why?
Using JSDoc type on
module.exports
doesn't work in TypeScript currently:create-next-app
already fixed this:How?
Create intermediate
nextConfig
variable and annotate that with JSDoc typePrior Art
Also done in
rspack-website
over here:cc @balazsorban44