scalameta/metals

Clean constructor parameters for `Indexer` and `ScalaCli`

Open

#4.788 geöffnet am 1. Jan. 2023

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Scala (421 Forks)batch import
good first issueimprovementtech debt

Repository-Metriken

Stars
 (2.300 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 3T 11h) (122 gemergte PRs in 30 T)

Beschreibung

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)

Contributor Guide

Research-Richtung
Identifizieren Sie Konstruktorparameter in Indexer und ScalaCli, die `() =>` verwenden, und ersetzen Sie sie durch direkte Werte oder andere Mechanismen, wobei sicherzustellen ist, dass der veränderliche Zustand korrekt behandelt wird.
Tech Stack
Keine
Domain
backend
Issue Type
Refactoring
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.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Scala
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70