Newspaper rule does not catch functions implemented above and used within a class

Métriques du dépôt

Stars

175 stars (175 stars)

Métriques de merge PR

Aucune PR mergée en 30 j (Aucune PR mergée en 30 j)

Description

function doStuff() {};

class StuffDoer {
  myMethod() {
    doStuff();
  }
}

class StuffDoer {
  myMethod() {
    doStuff();
  }
}

function doStuff() {};

Guide contributeur

Direction de recherche

Inspectez le walker AST de la règle newspaper et vérifiez pourquoi elle ne détecte que les fonctions déclarées en dessous des classes, pas au dessus. Regardez le code source de la règle pour voir si elle utilise un passage simple ou double.

Stack technique

typescript

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.

2

Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.

1-3 heures

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

TypeScriptTSLint rule development

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

70