dotnet/roslyn

Skip documents whose content failed to load from analysis

Ouverte

#41 661 ouverte le 13 févr. 2020

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-AnalyzersConcept-Continuous Improvementhelp wanted

Métriques du dépôt

Stars
 (20 414 étoiles)
Métriques de merge PR
 (Merge moyen 6j 17h) (256 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez comment Roslyn charge actuellement les documents pour l'analyse. La PR liée #40815 fournit le contexte de ce comportement. Recherchez où les analyseurs sont invoqués et assurez vous que les documents dont le chargement a échoué sont ignorés avant de passer l'arbre syntaxique aux analyseurs. Le changement implique probablement de modifier l'exécuteur d'analyseur ou le fournisseur de documents pour vérifier la réussite du chargement.
Stack technique
csharp
Domaine
backend
Type d'issue
Bug
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.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60