Glavin001/tslint-clean-code

Create a whitelist for id-length rule

Open

#40 aperta il 13 feb 2018

Vedi su GitHub
 (4 commenti) (0 reazioni) (1 assegnatario)TypeScript (15 fork)github user discovery
buggood first issuehelp wanted

Metriche repository

Star
 (175 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Hello and thank you for your work.

The id-length rule is great for forbidding one-letter variable names. However, there are cases where one-letter symbols would actually be preferred: type parameters (GenericTypeAnnotation nodes) and holes.

Holes

Array
  .from({ length: 10 })
  .map((_, index) => index);

Type parameters

function unique<T>(collection: T[]): T[] {
  return [...new Set(collection)];
}

What do you think about passing a config that would allow them?

"id-length": [true, {
  "check-type-parameters": false,
  "allow": ["_"]
}]

Guida contributor

Direzione di ricerca
Studia come altre regole TSLint gestiscono configurazioni simili (ad esempio, la regola 'variable name'). Esamina l'implementazione esistente della regola 'id length' e aggiungi proprietà di configurazione come 'allow' e 'check type parameters'.
Tech stack
typescript
Dominio
developer experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65