dotnet/roslyn

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

Aberta

#57.748 aberto em 13 de nov. de 2021

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)C# (4.257 forks)batch import
Area-IDEBugFeature - IDE0059help wanted

Métricas do repositório

Stars
 (20.414 estrelas)
Métricas de merge de PR
 (Mesclagem média 6d 17h) (256 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Rastreie o provedor de correção de código IDE0059 no código fonte do Roslyn para entender por que ele remove a atribuição em vez de introduzir um descarte e verifique se a descrição da correção é enganosa.
Pilha de tecnologia
csharp
Domain
compilersdeveloper experience
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70