Skip to content

Config linter does not take whitespace differences into account #3900

Open
@gonX

Description

@gonX

Seen in #3862 - PR provided a 4-wide indentation, but 2-wide indentation is expected:

{
Formatting = Formatting.Indented,
Indentation = 2
};

This likely happens on both 0.6.x and avalonia

DiffPlex.Diff has ignoreWhiteSpace = true as default - if we set this to false it'll likely fix the issues:
https://github.com/mmanela/diffplex/blob/b95d2e6da3cd185c3c6b0b50d715a36d781426ad/DiffPlex/DiffBuilder/InlineDiffBuilder.cs#L68

We should fix this relatively quickly before further bad configs are merged.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingneeds-triageThis issue or PR has not been properly labeled yetpriority:mediumMedium priority issue/pull requesttestsAdds or modifies a unit test

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions