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

Audit TypeScript Configuration #226

Merged
merged 4 commits into from
Dec 1, 2023
Merged

Audit TypeScript Configuration #226

merged 4 commits into from
Dec 1, 2023

Conversation

threeal
Copy link
Owner

@threeal threeal commented Dec 1, 2023

This pull request introduces the following changes to the tsconfig.json file:

  • Removes the following unnecessary compiler options:
    • moduleResolution, already defaults to the same as module option.
    • esModuleInterop, already defaults to true if using node16 or nodenext.
    • forceConsistentCasingInFileNames, already defaults to true.
    • lib, not required in this project.
  • Adds a exactOptionalPropertyTypes compiler option that set to true.
  • Use node16 module instead of nodenext.
  • Adjust module and target compiler options to use lowercase values.

It closes #225.

@threeal threeal self-assigned this Dec 1, 2023
@threeal threeal added the build label Dec 1, 2023
@threeal threeal added this to the Version 1.2.0 milestone Dec 1, 2023
@threeal threeal marked this pull request as ready for review December 1, 2023 15:11
@threeal threeal merged commit d976d24 into main Dec 1, 2023
4 checks passed
@threeal threeal deleted the audit-tsconfig branch December 1, 2023 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Audit TypeScript Configuration
1 participant