dotnet/roslyn

Change signature does not handle errors at call sites well

Aperta

#46.456 aperta il 30 lug 2020

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

Metriche repository

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

Descrizione

Version Used: Version 16.8.0 Preview 2.0 [30327.8.master]

Steps to Reproduce:

  1. Checkout Roslyn at a7976180954969c9904ee6d449adad28283ddc34
  2. Open src\Features\Core\Portable\SolutionCrawler\WorkCoordinator.WorkItem.cs and find WorkItem constructor on line 95
  3. Remove bool isLowPriority parameter using Change signature refactoring.
  4. Notice isLowPairority argument is now squiggled as expected. image
  5. Without fixing the error, use Change signature refactoring to remove bool isLowPriority parameter from the constructor on line 100.
  6. Review changes: image

Notice invocationReasons and retry: false were removed.

Guida contributor

Direzione di ricerca
Investiga il codice di refactoring ChangeSignature in Roslyn, concentrandoti su come gestisce i siti di chiamata con errori esistenti. Traccia la logica che enumera gli argomenti e identifica perché rimuove 'invocationReasons' e 'retry: false' quando è presente un errore.
Tech stack
csharp
Dominio
devtools
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
C#Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60