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

The fix for SA1312 can violate SA1300 #1425

Closed
Tinister opened this issue Sep 9, 2015 · 1 comment
Closed

The fix for SA1312 can violate SA1300 #1425

Tinister opened this issue Sep 9, 2015 · 1 comment

Comments

@Tinister
Copy link

Tinister commented Sep 9, 2015

See this example:

public class ClassName
{
    protected event EventHandler EventName;
}

This gets flagged as an SA1312 violation, and you are offered to make EventName camel case.

But once it's camel case, eventName is flagged as an SA1300 violation and you get offered to make it pascal case.

FYI, "old" Style Cop prefers this to be pascal case.

@sharwell
Copy link
Member

sharwell commented Sep 9, 2015

Duplicate of #1340.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants