dotnet/roslyn

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

Offen

#57.748 geöffnet am 13.11.2021

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C# (4.257 Forks)batch import
Area-IDEBugFeature - IDE0059help wanted

Repository-Metriken

Stars
 (20.414 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (256 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Verfolgen Sie den IDE0059 Codefix Anbieter im Roslyn Quellcode, um zu verstehen, warum er die Zuweisung entfernt, anstatt eine Verwerfung einzuführen, und überprüfen Sie, ob die Fix Beschreibung irreführend ist.
Tech Stack
csharp
Domain
compilersdeveloper experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70