dotnet/roslyn

Argument wrapping fails in the precense of comments

Ouverte

#40 368 ouverte le 13 déc. 2019

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-IDEBugIDE-CodeStylehelp wanted

Métriques du dépôt

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

Description

Consider this code:

public void M(object param1, object param2)
{
	M(
		1,
		// Comment
		2);
}

We do not show any help for adjusting how the comments are wrapped in this case. While I understand not being able to unwrap comments here (where are we supposed to put the comment, after all), changing the indentation level of the arguments should be possible. /cc @CyrusNajmabadi for thoughts on how this could be improved.

Guide contributeur

Direction de recherche
Examinez la logique d'encapsulation des arguments dans le moteur de formatage Roslyn pour comprendre comment les commentaires affectent l'indentation et l'encapsulation des arguments. Recherchez des tests liés à l'encapsulation des arguments avec des commentaires et tracez le chemin du code pour formater les arguments de méthode.
Stack technique
csharp
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
C#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75