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

Enable typechecking on config #57892

Merged
merged 9 commits into from
Dec 1, 2023
Merged

Enable typechecking on config #57892

merged 9 commits into from
Dec 1, 2023

Conversation

karlhorky
Copy link
Contributor

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 type

Prior Art

Also done in rspack-website over here:

cc @balazsorban44

@karlhorky karlhorky requested review from a team as code owners November 1, 2023 14:46
@karlhorky karlhorky requested review from ismaelrumzan and StephDietz and removed request for a team November 1, 2023 14:46
@ijjk ijjk added the examples Issue/PR related to examples label Nov 1, 2023
@ijjk
Copy link
Member

ijjk commented Nov 1, 2023

Allow CI Workflow Run

  • approve CI run for commit: 524b24f

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@ijjk
Copy link
Member

ijjk commented Nov 1, 2023

Allow CI Workflow Run

  • approve CI run for commit: 79aeb9d

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

balazsorban44
balazsorban44 previously approved these changes Nov 6, 2023
@karlhorky
Copy link
Contributor Author

@balazsorban44 thanks for the approval. I guess CI is currently broken in canary, can we merge this as-is?

@balazsorban44
Copy link
Member

The merging process is automatic. It might take a bit but will be merged.

@karlhorky
Copy link
Contributor Author

The merging process is automatic

ah nice, fancy. I hope I didn't break that automatic process by hitting the "Update branch" button to merge from canary (which created a new merge commit after your approval)

@karlhorky
Copy link
Contributor Author

@balazsorban44 it seems this still hasn't been automatically merged.

Should I do something? Or can you, from your side?

@karlhorky
Copy link
Contributor Author

@balazsorban44 Seems like the only thing that is failing now is the Datadog failure, seemingly unrelated to this PR:

Run actions/download-artifact@v3
Starting download for test-reports
Error: Unable to find an artifact with the name: test-reports

What do you think of ignoring the CI error and merging as is?

@huozhi huozhi merged commit b02a10e into vercel:canary Dec 1, 2023
53 of 54 checks passed
@karlhorky karlhorky deleted the patch-3 branch December 1, 2023 09:01
@karlhorky
Copy link
Contributor Author

Thanks for the review and merge @balazsorban44 @huozhi 🙌

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue/PR related to examples locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants