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.

Sign up for GitHub

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

Jump to bottom

csharpier-ignore comment not respected in certain scopes #1342

Closed
jamesfoster-excelpoint opened this issue Sep 4, 2024 · 1 comment · Fixed by #1345
Closed

csharpier-ignore comment not respected in certain scopes #1342

jamesfoster-excelpoint opened this issue Sep 4, 2024 · 1 comment · Fixed by #1345
Milestone
0.29.2

Comments

@jamesfoster-excelpoint
Copy link
Contributor

Input:

return new MyClass
{
    // csharpier-ignore-start
    MyVeryVeryVeryVeryVeryVeryLongProperty = someVeryVeryVeryVeryVeryVeryVeryLongVariable
    // csharpier-ignore-end
}

Output:

return new MyClass
{
    // csharpier-ignore-start
    MyVeryVeryVeryVeryVeryVeryLongProperty =
        someVeryVeryVeryVeryVeryVeryVeryLongVariable
    // csharpier-ignore-end
}

Expected behavior:

Respect the "ignore" comments inside of an object initialiser.
@belav belav added this to the 0.29.2 milestone Sep 8, 2024
belav added a commit that referenced this issue Sep 8, 2024
@belav
Adding support for csharpier-ignore-start in object initializers
dc00d0d 
closes #1342
@belav belav mentioned this issue Sep 8, 2024
Merged
@jamesfoster-excelpoint
Copy link
Contributor Author

jamesfoster-excelpoint commented Sep 11, 2024 •
edited
Loading

Thanks ❤️

@belav belav closed this as completed in 3fd7faa Sep 15, 2024
pisolofin pushed a commit to pisolofin/csharpier-editorconfig that referenced this issue Sep 24, 2024
@belav @pisolofin
Adding support for csharpier-ignore-start in object initializers (bel…
93160ce 
…av#1345)

closes belav#1342
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.29.2
Development

Successfully merging a pull request may close this issue.

Adding support for csharpier-ignore-start in object initializers belav/csharpier
2 participants
@belav @jamesfoster-excelpoint