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

Remove Default Values from tsconfig.json #665

Open
kevinfodness opened this issue Jun 26, 2023 · 1 comment
Open

Remove Default Values from tsconfig.json #665

kevinfodness opened this issue Jun 26, 2023 · 1 comment
Labels
enhancement New feature or request Package: tsconfig @alleyinteractive/tsconfig typescript Requires understanding TypeScript

Comments

@kevinfodness
Copy link
Member

Description

For all key/value pairs where the value we're specifying is already the default, remove the config option from tsconfig.json and rely on the default value instead.

Use Case

As a developer, I want to keep my configs lean and clean
@kevinfodness kevinfodness added the enhancement New feature or request label Jun 26, 2023
@srtfisher
Copy link
Member

We should also upgrade it to the shared standard: https://github.com/alleyinteractive/alley-scripts/blob/main/packages/tsconfig/README.md

@kevinfodness kevinfodness transferred this issue from alleyinteractive/create-wordpress-plugin Jul 10, 2024
@kevinfodness kevinfodness added Package: tsconfig @alleyinteractive/tsconfig typescript Requires understanding TypeScript labels Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request Package: tsconfig @alleyinteractive/tsconfig typescript Requires understanding TypeScript
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@srtfisher @kevinfodness