Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Change csharp_indent_labels from flush_left to one_less_than_current #28121

Merged
merged 2 commits into from
Mar 17, 2018

Conversation

ahsonkhan
Copy link
Member

@ahsonkhan ahsonkhan commented Mar 15, 2018

Most of the places where we use goto and labels (that I have come across) use one_less_than_current formatting pattern.

I will make the same change in coreclr as well - dotnet/coreclr#16983

cc @jkotas, @danmosemsft, @cod7alex, @karelz, @tannergooding

@ahsonkhan ahsonkhan self-assigned this Mar 15, 2018
@joshfree joshfree requested a review from stephentoub March 16, 2018 00:56
@ahsonkhan
Copy link
Member Author

@dotnet-bot test OSX x64 Debug Build

@tannergooding
Copy link
Member

@ahsonkhan, can we update the coding style guidelines to call this out as well?

Otherwise, LGTM

@ahsonkhan ahsonkhan merged commit 5f5ff6d into dotnet:master Mar 17, 2018
@ahsonkhan ahsonkhan deleted the UpdateEditorconfig branch March 17, 2018 01:40
@karelz karelz added this to the 2.1.0 milestone Mar 18, 2018
ericstj pushed a commit to ericstj/corefx that referenced this pull request Mar 28, 2018
…otnet#28121)

* Change csharp_indent_labels from flush_left to one_less_than_current

* Update coding-style guidelines to include label formatting.
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…otnet/corefx#28121)

* Change csharp_indent_labels from flush_left to one_less_than_current

* Update coding-style guidelines to include label formatting.


Commit migrated from dotnet/corefx@5f5ff6d
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants