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

Métricas do repositório

Stars

175 stars (175 stars)

Métricas de merge de PR

Nenhuma PRs mesclada em 30d (Nenhuma PRs mesclada em 30d)

Description

function doStuff() {};

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

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

function doStuff() {};

Guia do colaborador

Direção de pesquisa

Inspecione o walker AST da regra newspaper e verifique por que ela detecta apenas funções declaradas abaixo das classes, não acima. Veja o código fonte da regra para saber se usa uma travessia única ou dupla.

Pilha de tecnologia

typescript

Domain

tooling

Tipo Issue

Bug

DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.

2

Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.

1-3 horas

Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.

Recente

ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.

Claro

Prerequisites

TypeScriptTSLint rule development

Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.

70