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

SA1004 should report a warning for indented top-level elements #1238

Closed
sharwell opened this issue Aug 17, 2015 · 0 comments
Closed

SA1004 should report a warning for indented top-level elements #1238

sharwell opened this issue Aug 17, 2015 · 0 comments
Assignees
Milestone

Comments

@sharwell
Copy link
Member

Currently SA1004 does not report a warning for the following code:

/// <summary>
/// The summary text.
/// </summary>
///   <param name="x">The document root.</param>
///    <param name="y">The Xml header token.</param>
private void Method1(int x, int y)
{
}

According to the documentation for this rule, it should report a warning here. The exclusion for indented lines should only apply to content, such as the following:

/// <summary>
///     The summary text.
/// </summary>
/// <param name="x">The document root.</param>
/// <param name="y">The Xml header token.</param>
private void Method1(int x, int y)
{
}
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

1 participant