dotnet/roslyn

Change signature does not handle errors at call sites well

Ouverte

#46 456 ouverte le 30 juil. 2020

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-IDEBughelp 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

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.

Guide contributeur

Direction de recherche
Étudiez le code de refactoring ChangeSignature dans Roslyn, en vous concentrant sur la manière dont il gère les sites d'appel avec des erreurs existantes. Suivez la logique qui énumère les arguments et identifiez pourquoi il supprime 'invocationReasons' et 'retry: false' lorsqu'une erreur est présente.
Stack technique
csharp
Domaine
devtools
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.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60