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

Repository-Metriken

Stars

175 Stars (175 Stars)

PR-Merge-Metriken

Keine gemergten PRs in 30 T (Keine gemergten PRs in 30 T)

Beschreibung

function doStuff() {};

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

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

function doStuff() {};

Contributor Guide

Research-Richtung

Überprüfe den AST Walker der Newspaper Regel und stelle fest, warum nur Funktionen unterhalb von Klassen erkannt werden, nicht oberhalb. Sieh dir den Quellcode der Regel an, um zu sehen, ob ein Single Pass oder Two Pass verwendet wird.

Tech Stack

typescript

Domain

tooling

Issue Type

Fehler

SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.

2

Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.

1-3 Stunden

AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.

Frisch

KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.

Klar

Voraussetzungen

TypeScriptTSLint rule development

EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.

70