dotnet/roslyn

Change signature removes parameter I didn't select to be removed

Open

#62.824 aperta il 21 lug 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-IDEBughelp wanted

Metriche repository

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

Descrizione

Using https://github.com/dotnet/roslyn/commit/deabaa7744075fec2b2688872576561259761552

Go to https://github.com/dotnet/roslyn/blob/deabaa7744075fec2b2688872576561259761552/src/Analyzers/Core/Analyzers/AbstractBuiltInCodeStyleDiagnosticAnalyzer.cs#L72

Invoke "Change signature" and remove the unused language parameter. Note this strange change:

image

It also uses named arguments unnecessarily, but that's a minor point

image

Guida contributor

Direzione di ricerca
Individua il codice del refactoring 'Change Signature' nel repository Roslyn, in particolare il metodo che gestisce la rimozione dei parametri. Segui la logica per capire perché rimuove anche un parametro non selezionato. Esamina i casi di test per Change Signature per riprodurre il problema.
Tech stack
csharp
Dominio
tooling
Tipo issue
Bug
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
GitC#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60