Glavin001/tslint-clean-code

Create a whitelist for id-length rule

Open

#40 ouverte le 13 févr. 2018

Voir sur GitHub
 (4 commentaires) (0 réactions) (1 assigné)TypeScript (15 forks)github user discovery
buggood first issuehelp wanted

Métriques du dépôt

Stars
 (175 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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": ["_"]
}]

Guide contributeur