dotnet/roslyn

Move to new file doesn't honor charset in .editorconfig

Offen

#34.200 geöffnet am 17.03.2019

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)C# (4.257 Forks)batch import
Area-IDEBugFeature - Editor Confighelp wanted

Repository-Metriken

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

Beschreibung

Version Used: 15.9.9

Steps to Reproduce:

  1. Have an .editorconfig with charset = utf-8
  2. Open an existing file (encoding irrelvant)
  3. Add a type to the file
  4. Apply "Move type to new file" refactoring

Expected Behavior:

The new file is stored in the encoding specified by .editorconfig (in this case UTF8, no BOM).

Actual Behavior:

The new file is stored in UTF8-BOM.

Contributor Guide

Research-Richtung
Untersuche die Refactoring Aktion 'Typ in neue Datei verschieben' in der Roslyn Codebasis. Finde heraus, wo die neue Datei erstellt wird und wie die Kodierung bestimmt wird. Überprüfe, ob der .editorconfig Zeichensatz gelesen und angewendet wird.
Tech Stack
csharp
Domain
developer 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
GitC#
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60