eclipsesource / eclipsesource/jsonforms

Increase strictness of `tsconfig`

Aperta
#2,255 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
dev help wanted
Lingua principale
TypeScript
Stelle
2.7k
Fork
424
Merge medio
17g 8h
PR unite (30g)
1

Descrizione

### Is your feature request related to a problem? Please describe.

Currently `tsconfig.base.json` has quite low strictness, and Typescript is not helping the developer as much as it could.

Also, there is a higher change of bugs arising without this strictness.

I suggest `"strict": true` and `"noUncheckedIndexedAccess": true` is added to `tsconfig`, but there are also some other settings that could be interesting.

Matt Pocock has a good blog post here: [https://www.totaltypescript.com/tsconfig-cheat-sheet](https://www.totaltypescript.com/tsconfig-cheat-sheet).

In case this is something that could be added, we should start by updating `@jsonforms/core` and could afterwards do other packages.

It would require quite a bit of (small) changes several places.

### Describe the solution you'd like

`packages/core/tsconfig.json`:
```jsonc
{
"compilerOptions": {
"verbatimModuleSyntax": true,
/* strictness */
"strict": true,
"noUncheckedIndexedAccess": true,
}
}
```

see also:
- [strict](https://www.typescriptlang.org/tsconfig#strict)
- [noUncheckedIndexedAccess](https://www.typescriptlang.org/tsconfig#noUncheckedIndexedAccess)

### Describe alternatives you've considered

.

### Framework

Core

### RendererSet

_No response_

### Additional context

_No response_

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia leggendo tsconfig.base.json e packages/core/tsconfig.json, concentrandoti sulle opzioni di strictness proposte per @jsonforms/core. Esamina gli errori TypeScript prodotti dopo aver abilitato strict e noUncheckedIndexedAccess, quindi apporta le piccole modifiche necessarie in tutto il package. Il lavoro è completo quando il package core viene compilato correttamente con entrambe le opzioni abilitate.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
typescript
Ambito
build-system, developer-experience
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.