mysticatea/eslint-plugin-node

Rule to ensure `process.env.NODE_ENV` is only equality-tested against valid values

Open

#101 ouverte le 8 déc. 2017

Voir sur GitHub
 (1 commentaire) (3 réactions) (0 assignés)JavaScript (186 forks)github user discovery
acceptedenhancementhelp wanted

Métriques du dépôt

Stars
 (956 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

I want to make sure that no one accidentally uses a condition in their code like process.env.NODE_ENV !== 'productn'.

The rule would error if process.env.NODE_ENV is equality-tested, or inequality-tested, against a value that is not a literal string of one of the allowed values. Allowed values could default to the conventional 'development' and 'production', and could be configured by the user.

Thoughts? Should I go ahead with a PR?

Guide contributeur

Direction de recherche
Examinez les règles existantes dans eslint plugin node pour comprendre le modèle de création d'une nouvelle règle. Implémentez ensuite une règle qui vérifie les expressions binaires comparant process.env.NODE ENV à des littéraux de chaîne et assurez vous que seules les valeurs autorisées sont utilisées.
Stack technique
javascriptnodejs
Domaine
toolingbackend
Type d'issue
Fonctionnalité
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.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
JavaScriptNode.jsESLint
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65