dotnet/roslyn

Incorrect indentation in LambdaRewriter

Open

#12.709 geöffnet am 24. Juli 2016

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (4.257 Forks)batch import
Area-CompilersBughelp wanted

Repository-Metriken

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

Beschreibung

While looking at LambdaRewriter.cs, I noticed that some lines are incorrectly indented. Specifically, it's the lines fixed by this commit in my fork.

As far as I can tell, the incorrect changes are from this merge from September 2015.

I haven't checked if other files have similar problems.

The Contributing Code page in the repo wiki says that Code Formatter is regularly run on the code in the repo. Presumably, running it would fix the indentation, so I guess it hasn't been run at least since September 2015?

My questions are: Should I just submit a PR with the commit above? Or should someone run the Code Formatter?

Contributor Guide

Research-Richtung
Zeigen Sie die Datei LambdaRewriter.cs im angegebenen Pfad an, vergleichen Sie sie mit dem Commit im Fork, um die genauen Einrückungsänderungen zu sehen, und wenden Sie dann die Korrektur an, indem Sie die Datei bearbeiten und bei Bedarf den Code Formatter ausführen.
Tech Stack
csharp
Domain
compilers
Issue Type
Wartung
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
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
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90