Skip to content
/ SA1413 Public

Use Trailing Commas In MultiLine Initializers Analyzer

License

Notifications You must be signed in to change notification settings

krk/SA1413

Repository files navigation

SA1413 https://github.com/DotNetAnalyzers/StyleCopAnalyzers rule as a stand-alone Roslyn Analyzer.

Build status NuGet

SA1413

TypeName SA1413UseTrailingCommasInMultiLineInitializers
CheckId SA1413
Category Maintainability Rules

📝 This rule is new for StyleCop Analyzers, and was not present in StyleCop Classic.

Cause

The last statement in a multi-line C# initializer or list is missing a trailing comma.

Rationale

This rule is specifically designed to work well with the most widely used source control systems as an aid to long-term code review. By placing a comma on the last line of a multi-line sequence, developers who append an item to the list or reorder the list at some point in the future will not need to modify any more lines than absolutely necessary for the change. As a result, the size of the subsequent code review is minimized and focused, and tools like git blame continue to show the original author and commit for the item that was previously last in the list.

Rule description

A violation of this rule occurs when the last statement of a C# initializer or list is missing a trailing comma.

For example, the following code would generate one instance of this violation:

var x = new Barnacle
{
    Age = 100,
    Height = 0.2M,
    Weight = 0.88M
};

The following code would not produce any violations:

var x = new Barnacle
{
    Age = 100,
    Height = 0.2M,
    Weight = 0.88M,
};

This diagnostic is also reported for other forms of comma-separated list, such as enum members.

How to fix violations

To fix a violation of this rule, add a trailing comma to the last statement in the initializer.

How to suppress violations

[SuppressMessage("StyleCop.CSharp.MaintainabilityRules", "SA1413:UseTrailingCommasInMultiLineInitializers", Justification = "Reviewed.")]
#pragma warning disable SA1413 // UseTrailingCommasInMultiLineInitializers
#pragma warning restore SA1413 // UseTrailingCommasInMultiLineInitializers

About

Use Trailing Commas In MultiLine Initializers Analyzer

Resources

License

Stars

Watchers

Forks

Releases

No releases published

Packages

No packages published