dotnet/roslyn

Remove the IOptionProvider and ExportGlobalOptionProvider from FeatureOnOffOptions

Aperta

#60.082 aperta il 10 mar 2022

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-IDEConcept-Continuous Improvementhelp wanted

Metriche repository

Star
 (20.414 stelle)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

You can delete these. In fact, you can remove the IOptionProvider and ExportGlobalOptionProvider from this class entirely since these are all global options and as such they don't need IOptionProvider

Originally posted by @tmat in https://github.com/dotnet/roslyn/pull/60053#discussion_r823064107

Guida contributor

Direzione di ricerca
Individua la classe `FeatureOnOffOptions` nella codebase di Roslyn. Il problema fa riferimento a una discussione nella PR #60082, quindi esaminare quella PR e il commento collegato potrebbe fornire contesto. Rimuovi le interfacce `IOptionProvider` e `ExportGlobalOptionProvider` dalla classe, poiché sono opzioni globali non necessarie. Assicurati che nessun'altra parte del codice dipenda da queste interfacce.
Tech stack
csharp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80