scalameta/metals

Clean constructor parameters for `Indexer` and `ScalaCli`

Open

#4 788 ouverte le 1 janv. 2023

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)Scala (421 forks)batch import
good first issueimprovementtech debt

Métriques du dépôt

Stars
 (2 300 stars)
Métriques de merge PR
 (Merge moyen 3j 11h) (122 PRs mergées en 30 j)

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)

Guide contributeur

Direction de recherche
Identifiez les paramètres du constructeur dans Indexer et ScalaCli qui utilisent `() =>` et remplacez les par des valeurs directes ou d'autres mécanismes, en veillant à ce que l'état mutable soit correctement géré.
Stack technique
Aucun
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Scala
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70