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

Always break statements without braces. #303

Closed
belav opened this issue Jun 12, 2021 · 1 comment · Fixed by #306
Closed

Always break statements without braces. #303

belav opened this issue Jun 12, 2021 · 1 comment · Fixed by #306
Labels
area:formatting
Milestone
0.9.7

Comments

@belav
Copy link
Owner

belav commented Jun 12, 2021

Statements that have optional braces should always break. I believe they are

  • if
  • else
  • while
  • foreach
  • for
if (behaviors != null) foreach (var behavior in behaviors) AddBehavior(behavior);

// should be
if (behaviors != null)
    foreach (var behavior in behaviors)
        AddBehavior(behavior);
@belav belav added this to the 0.9.7 milestone Jun 12, 2021
@belav
Copy link
Owner Author

belav commented Jun 12, 2021

This will close #245 and #255

belav added a commit that referenced this issue Jun 14, 2021
@belav
Always break + indent with optional braces
1ea72ee 
closes #303
closes #245
closes #255
@belav belav mentioned this issue Jun 14, 2021
Merged
belav added a commit that referenced this issue Jun 16, 2021
@belav
Always break + indent with optional braces (#306)
1cc3a5f 
closes #303
closes #245
closes #255
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:formatting
Projects
None yet
Milestone
0.9.7
Development

Successfully merging a pull request may close this issue.

Always break + indent with optional braces belav/csharpier
1 participant
@belav