jsx-eslint/eslint-plugin-react

[ESLint v5.9.0] Fix types

Ouverte

#2 039 ouverte le 9 nov. 2018

Voir sur GitHubGuide contributeur
 (2 commentaires) (1 réaction) (0 personne assignée)JavaScript (2 731 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (9 293 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

A new option was added to ESLint. Reference: https://eslint.org/blog/2018/11/eslint-v5.9.0-released#the-fix-type-option

Should we start supporting this? If so, looks like "layout" is pretty obvious, but "problem" vs "suggestion" could be harder to tell. Or should we just set all recommended non-layout rules to "problem"?

Guide contributeur

Direction de recherche
Consultez les notes de version d'ESLint v5.9.0 pour l'option de type de correction (https://eslint.org/blog/2018/11/eslint v5.9.0 released#the fix type option). Examinez les règles existantes dans le plugin (par exemple dans src/rules/) pour comprendre les valeurs actuelles de meta.fixable. Déterminez une stratégie pour catégoriser les règles en 'layout', 'problem' ou 'suggestion' en fonction de l'intention de la règle. Décidez s'il faut définir toutes les règles recommandées non liées à la mise en page sur 'problem' ou si une analyse par règle est nécessaire. Implémentez les modifications de la propriété meta.fixable pour chaque règle et ajoutez les mises à jour de test correspondantes.
Stack technique
javascriptreact
Domaine
developer experiencetoolingfrontend
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.
Claire
Prérequis
ESLintReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60