Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Comments are not automatically wrapped. #1352

Open
make1980 opened this issue Sep 24, 2024 · 4 comments · May be fixed by #1355
Open

Comments are not automatically wrapped. #1352

make1980 opened this issue Sep 24, 2024 · 4 comments · May be fixed by #1355
Assignees

Comments

@make1980
Copy link

make1980 commented Sep 24, 2024

Single line/multi line comments are not automatically wrapped/checked by csharpier.

Input:

// Test very long line comment.
// Second line of long comment.
var someVariable = someValue;

Width = 10

Output:

// Test very long line comment.
// Second line of long comment.
var someVariable = someValue;

Expected behavior:

// Test
// very
// long
// line
// comment.
// Second
// line
// of
// long
// comment.
var someVariable =
    someValue;
@belav
Copy link
Owner

belav commented Sep 24, 2024

I'm not opposed to this, but it may be a lot of work. Prettier has had the request for 7 years now - prettier/prettier#265

That also brings up some points about potential issues. ascii diagrams, table-formatted text, etc.

@make1980
Copy link
Author

make1980 commented Sep 26, 2024

I have a PR for this - will publish to get feedbacks. #1355

@make1980
Copy link
Author

.assign

@make1980 make1980 linked a pull request Sep 26, 2024 that will close this issue
@Hona
Copy link

Hona commented Oct 23, 2024

I feel like this would break far too many things, than the few things it would help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants