dotnet/roslyn

IDE0059 and code fix for use discard but instead (correctly) simplifies to type match

Ouverte

#57 748 ouverte le 13 nov. 2021

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-IDEBugFeature - IDE0059help 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: .NET 6.0.100

Steps to Reproduce:

  1. Have a type match like that Foo foo => throw new NotImplementedException(),
  2. Code Analyzers correctly detect IDE0059 unnecessary assignment
  3. Code Fixes suggest to use discard (not wrong but so C# 8?)
  4. Code Fixes executes and emits the following ... Foo => throw new NotImplementedException(),

Expected Behavior: Either, Code Fix does what it says in the UI (introduce a discard) OR is correctly named/described.

Actual Behavior: Code Fix does something else then what it actually says. Not that the refactoring itself is wrong ... i am happy with the overall outcome but the description meanwhile is misleading.

Guide contributeur

Direction de recherche
Tracez le fournisseur de correction de code IDE0059 dans le code source de Roslyn pour comprendre pourquoi il supprime l'affectation au lieu d'introduire un discard, et vérifiez si la description de la correction est trompeuse.
Stack technique
csharp
Domaine
compilersdeveloper experience
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.
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.
70