jshint/jshint

Should extends merge predef arrays?

Open

#2 177 ouverte le 16 févr. 2015

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (1 757 forks)batch import
P4cli Issuedeprecatedgood first issue

Métriques du dépôt

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

Description

I have a .jshintrc in my top level folder with:

{
  "predef"    : [
    "-Promise"
  ]
}

and one in my test folder with:

{
  "extends": "../.jshintrc",
  "predef": [
    "describe",
    "it",
    "before",
    "after",
    "beforeEach",
    "afterEach"
  ]
}

But it seems in my testfolder the -Promise is not recognized. Does it make sense to merge these predef configurations when extending .jshintrc?

Guide contributeur

Direction de recherche
Étudiez la logique de fusion de configuration de JSHint, probablement dans le fichier source gérant l'analyse de .jshintrc (par exemple, src/config.js). Reproduisez le scénario décrit dans l'issue pour confirmer que les tableaux predef sont remplacés plutôt que fusionnés lors de l'utilisation de extends. Déterminez le comportement attendu et proposez une modification pour fusionner les tableaux (par exemple, en utilisant concat ou similaire) tout en respectant les marqueurs de négation comme ' Promise'. Recherchez les tests existants liés à extends pour garantir la rétrocompatibilité.
Stack technique
javascript
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptJSON
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60