scalameta/metals

Clean constructor parameters for `Indexer` and `ScalaCli`

Open

#4.788 aberto em 1 de jan. de 2023

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)Scala (421 forks)batch import
good first issueimprovementtech debt

Métricas do repositório

Stars
 (2.300 stars)
Métricas de merge de PR
 (Mesclagem média 3d 11h) (122 fundiu PRs em 30d)

Description

Another follow up after https://github.com/scalameta/metals/pull/4776.

There's need to pass every constructor argument as a function in scala.meta.internal.metals.Indexer and scala.meta.internals.ScalaCli

  • Usually, replacing () =>, should be straighforward change, but sometimes () => was used to capture mutable state.
  • probably Indexerand ScalaCli aren't the only ones which can be refactored. Person doing this task can browse other classes (optional)

Guia do colaborador

Direção de pesquisa
Identifique os parâmetros do construtor em Indexer e ScalaCli que usam `() =>` e substitua os por valores diretos ou outros mecanismos, garantindo que o estado mutável seja tratado corretamente.
Pilha de tecnologia
None
Domain
backend
Tipo Issue
Refatoração
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Scala
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70