dotnet/roslyn

AbstractSuppressionCodeFixProvider replaces LF with CRLF

Open

#69.358 aperta il 3 ago 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-IDEBughelp wanted

Metriche repository

Star
 (20.414 star)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

Version Used: Version 17.8.0 Preview 1.0 [33916.57.main]

Steps to Reproduce:

This issue tracks multiple disabled tests in code.

  1. Open a file with LF line endings in a project with end_of_line unset (Windows only) or explicitly set to CRLF
  2. Place the caret on a line with a diagnostic
  3. Suppress the diagnostic via the light bulb with a #pragma warning disable or #Disable Warning

Expected Behavior:

The file retains its LF line endings, including on the two newly inserted lines.

Actual Behavior:

The file has mixed line endings.

Guida contributor

Direzione di ricerca
Indaga sulla classe AbstractSuppressionCodeFixProvider, probabilmente in src/Features/Core/Portable/CodeFixes/Suppression/. Esamina come inserisce le direttive #pragma e se scrive le righe utilizzando un metodo che rispetti le terminazioni di riga originali. Controlla i test disabilitati menzionati nell'issue per comprendere il comportamento atteso. Cerca eventuali test esistenti che verificano la conservazione delle terminazioni di riga e identifica dove dovrebbe essere applicata la correzione.
Tech stack
csharp
Dominio
developer experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git.NET SDK
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60