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

Metriche repository

Star

175 star (175 star)

Metriche merge PR

Nessuna PR mergiata in 30 g (Nessuna PR mergiata in 30 g)

Descrizione

function doStuff() {};

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

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

function doStuff() {};

Guida contributor

Direzione di ricerca

Ispeziona il walker AST della regola newspaper e verifica perché rileva solo le funzioni dichiarate sotto le classi, non sopra. Guarda il sorgente della regola per vedere se utilizza una traversata singola o doppia.

Tech stack

typescript

Dominio

tooling

Tipo issue

Bug

DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.

2

Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.

1-3 ore

Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.

Nuova

ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.

Chiara

Prerequisiti

TypeScriptTSLint rule development

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

70