dotnet/roslyn

Incorrect indentation in LambdaRewriter

Open

#12 709 ouverte le 24 juil. 2016

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)C# (4 257 forks)batch import
Area-CompilersBughelp wanted

Métriques du dépôt

Stars
 (20 414 stars)
Métriques de merge PR
 (Merge moyen 6j 17h) (256 PRs mergées en 30 j)

Description

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?

Guide contributeur

Direction de recherche
Affichez le fichier LambdaRewriter.cs au chemin spécifié, comparez avec le commit dans le fork pour voir les changements exacts d'indentation, puis appliquez la correction en éditant le fichier et en exécutant le formateur de code si nécessaire.
Stack technique
csharp
Domaine
compilers
Type d'issue
Maintenance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90