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

Configure TypeScript config rule by file name #49

Open
uPaymeiFixit opened this issue Oct 16, 2024 · 0 comments
Open

Configure TypeScript config rule by file name #49

uPaymeiFixit opened this issue Oct 16, 2024 · 0 comments
Labels
p2 Priority 2 (Medium) spike

Comments

@uPaymeiFixit
Copy link
Collaborator

image

There’s 4 routes we could go down here:

  1. Use multiple tsconfig.json files
  • ❌ complicates build
  • ❌ adds a whole other config file for each rule we want to disable
  1. Use multiple tsconfig.json files with project references (see Convert to tsconfig's project references · Issue #18 · dudeofawesome/code-style)
  • ❌ complicates build
  • ❌ not supported by some of our tools yet
  1. Use something like ESLint’s overrides in tsconfig.json (not supported at this time Allow overrides in tsconfig · Issue #33407 · microsoft/TypeScript)
  • ❌ not actually a thing
  1. Turn off the offending strict rule in typescript and ban the syntax with ESLint
  • ❌ duplicates rules between TS & ESLint
  • ✅ minimizes config files
  • ✅ keeps build simple
@uPaymeiFixit uPaymeiFixit added p2 Priority 2 (Medium) spike labels Oct 16, 2024
@dudeofawesome dudeofawesome moved this to Backlog in code-style Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p2 Priority 2 (Medium) spike
Projects
Status: Backlog
Development

No branches or pull requests

1 participant