-
-
Notifications
You must be signed in to change notification settings - Fork 91
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
Empty Method should keep braces on same line #133
Labels
Milestone
Comments
Closed
I think that's fine - making a special rule to go past the character limit
for this doesn't seem worth it.
…On Fri, Apr 23, 2021 at 4:06 PM Bela VanderVoort ***@***.***> wrote:
This used to keep the { } on the same line. No need to break if there is
nothing actually in the body.
public virtual Castle.Components.DictionaryAdapter.IDictionaryBehavior Copy()
{ }
On the other hand this is past the 80 character limit, and doesn't break
at all til the braces. But maybe that is fine?
public override System.Collections.Generic.IEnumerable<string> GetDynamicMemberNames()
{ }
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#133>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGRDLN6ETZVR2TRUCNECCTTKH4QDANCNFSM43POZQIQ>
.
|
belav
added a commit
that referenced
this issue
Jul 19, 2021
belav
added a commit
that referenced
this issue
Jul 23, 2021
* Make sure empty block doesn't break on method declaration closes #133 * drinking the kool-aid
belav
changed the title
Empty Method now breaks
Empty Method should keep braces on same line
Jul 27, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This used to keep the { } on the same line. No need to break if there is nothing actually in the body.
On the other hand this is past the 80 character limit, and doesn't break at all til the braces. But maybe that is fine?
The text was updated successfully, but these errors were encountered: