Tslint: L'écrasement des options de règle ne fonctionne pas

Créé le 20 juil. 2017 · 3Commentaires · Source: palantir/tslint

Rapport d'erreur

__Version TSLint__ : 5.4.3
__TypeScript version__ : 2.3.4
__Exécution de TSLint via__ : CLI

Code TypeScript en train d'être lint

console.log()

avec la configuration tslint.json :

{
  "extends": "tslint-microsoft-contrib",
  "rules": {
    "no-console": {
      "options": [
        "time",
        "timeEnd"
      ]
    }
  }
}

tslint-microsoft-contrib :

    "no-console": [
      true,
      "debug",
      "info",
      "log",
      "time",
      "timeEnd",
      "trace"
    ],

Comportement réel

N'échoue pas pour console.log.

Comportement prévisible

Devrait échouer.

Not A Bug

Source

CSchulz

Commentaire le plus utile

@CSchulz #924 contient déjà quelques réflexions sur ce sujet. Cette discussion devrait se poursuivre là-bas.

ajafff le 22 août 2017

👍2

Tous les 3 commentaires

Les options de cette règle sont une liste noire, pas une liste blanche.
Si vous ne spécifiez aucune méthode interdite, toutes sont interdites par défaut. Si vous spécifiez des options, seules ces méthodes sont interdites.

Nous devrions certainement ajouter la possibilité de mettre certaines méthodes en liste blanche au lieu de mettre en liste noire toutes les autres méthodes. Mais c'est hors de portée pour ce problème.

ajafff le 21 juil. 2017

J'aimerais l'implémenter en tant que liste blanche, mais je ne vois aucun bon moyen de l'implémenter sans casser les modifications.

Avez-vous un indice pour moi?

CSchulz le 22 août 2017

@CSchulz #924 contient déjà quelques réflexions sur ce sujet. Cette discussion devrait se poursuivre là-bas.

ajafff le 22 août 2017

👍2

Cette page vous a été utile?

0 / 5 - 0 notes