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

Formatting of auto implemented properties with access modifiers #188

Closed
respel opened this issue May 12, 2021 · 1 comment · Fixed by #199
Closed

Formatting of auto implemented properties with access modifiers #188

respel opened this issue May 12, 2021 · 1 comment · Fixed by #199
Labels
area:formatting type:bug Something isn't working
Milestone

Comments

@respel
Copy link

respel commented May 12, 2021

//Is
public class ClassName {
    public string ShortPropertyName {
        get;
        private set; 
    }
}
//ShouldBe
public class ClassName {
    public string ShortPropertyName { get; private set; }
}

The latter seems more idiomatic

@belav belav added type:bug Something isn't working area:formatting labels May 12, 2021
@belav
Copy link
Owner

belav commented May 12, 2021

I don't recall if I intentionally set up the breaking with access modifiers but if everything can fit within the print width I think it makes sense to keep it on a single line.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:formatting type:bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants