dotnet/roslyn

Skip documents whose content failed to load from analysis

Aperta

#41.661 aperta il 13 feb 2020

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

Metriche repository

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

Descrizione

Analyzers shouldn't be run on documents (syntax trees) with load failures. These should be treated as generated files (they are empty).

Follow up on https://github.com/dotnet/roslyn/pull/40815.

Guida contributor

Direzione di ricerca
Esamina come Roslyn attualmente carica i documenti per l'analisi. La PR collegata #40815 fornisce il contesto per questo comportamento. Indaga dove vengono richiamati gli analizzatori e assicurati che i documenti con errori di caricamento vengano saltati prima di passare l'albero sintattico agli analizzatori. La modifica probabilmente comporta la modifica dell'esecutore dell'analizzatore o del provider di documenti per verificare il successo del caricamento.
Tech stack
csharp
Dominio
backend
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.
Nuova
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