Skip to content

Conversation

@marcalexiei
Copy link
Contributor

@marcalexiei marcalexiei commented Dec 14, 2024

Part of #1098.

Note

Based upon #1102 (I marked this PR as draft, will rebase and make it ready for review after #1102 is merged).

  • fixed typescript errors
  • added typecheck script to ci.yml checks
  • added typecheck:ci script to ci.yml checks
    I'm not using directly typecheck script because src/pages/[...slug].tsx file requires types from contentlayer/generated which are available only after next build.
    To resolve this issue I created a new typecheck:ci script that runs next build --no-lint before tsc --noEmit
    Let me know if you want that this explanation should be added to CONTRIBUTING.md for future reference.

@vercel
Copy link

vercel bot commented Dec 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-hook-form-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 15, 2024 7:02am

Copy link
Member

@bluebill1049 bluebill1049 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks again.

@bluebill1049 bluebill1049 merged commit 4b4b39d into react-hook-form:master Dec 15, 2024
5 checks passed
@marcalexiei marcalexiei deleted the typecheck branch December 15, 2024 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants