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

Use TypeScript for the React apps #2314

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Use TypeScript for the React apps #2314

wants to merge 11 commits into from

Conversation

tomodwyer
Copy link
Member

@tomodwyer tomodwyer commented Jan 21, 2025 •
edited
Loading

Closes #2249

  • Set up TypeScript with Vite
  • Add types to React app
  • Fix issues highlighted by TypeScript checks
  • Check TypeScript types as part of the CI and linting process
  • Remove unnecessary prop-types package
  • Add extra tests using TypeScript to confirm Vitests works with TypeScript
  • Remove untested JS files from code coverage count

@tomodwyer tomodwyer self-assigned this Jan 21, 2025
@tomodwyer tomodwyer force-pushed the typescript-setup branch 4 times, most recently from c73f9ae to 6b7c3f4 Compare January 22, 2025 10:39
@tomodwyer tomodwyer marked this pull request as ready for review January 22, 2025 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tomodwyer tomodwyer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Use TypeScript for the Codelist Builder app
1 participant
@tomodwyer