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

Remove isEnabledByDefault argument when it's true #537

Closed
Vannevelj opened this issue Apr 26, 2016 · 1 comment
Closed

Remove isEnabledByDefault argument when it's true #537

Vannevelj opened this issue Apr 26, 2016 · 1 comment
Labels
priority - low status - wontfix type - task

Comments

@Vannevelj
Copy link
Owner

Several analyzers have isEnabledByDefault: true as part of their DiagnosticDescriptor rule. This can be removed considering analyzers are enabled by default.
@Vannevelj
Copy link
Owner Author

False alarm. These are named arguments, not optional arguments. They're written in this named manner to make it clear what the truevalue actually means.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority - low status - wontfix type - task
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Vannevelj