scalameta/metals

Clean constructor parameters for `Indexer` and `ScalaCli`

Open

#4788 aperta il 1 gen 2023

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Scala (421 fork)batch import
good first issueimprovementtech debt

Metriche repository

Star
 (2300 star)
Metriche merge PR
 (Merge medio 3g 11h) (122 PR mergiate in 30 g)

Descrizione

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)

Guida contributor

Direzione di ricerca
Identifica i parametri del costruttore in Indexer e ScalaCli che usano `() =>` e sostituiscili con valori diretti o altri meccanismi, assicurandoti che lo stato mutabile sia gestito correttamente.
Tech stack
Nessuno
Dominio
backend
Tipo issue
Refactoring
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Scala
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70