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

Update CSharpier to v0.28.2 #114

Merged
merged 1 commit into from
May 1, 2024
Merged

Update CSharpier to v0.28.2 #114

merged 1 commit into from
May 1, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Apr 7, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
CSharpier.MsBuild 0.27.3 -> 0.28.2 age adoption passing confidence
csharpier 0.27.3 -> 0.28.2 age adoption passing confidence

Release Notes

belav/csharpier (CSharpier.MsBuild)

v0.28.2

Compare Source

What's Changed

Pipe to dotnet csharpier fails when subdirectory is inaccessible #​1240

When running the following CSharpier would look for config files in subdirectories of the pwd. This could lead to exceptions if some of those directories were inaccessible.

echo "namespace Foo { public class Bar { public string Baz {get;set;}}}" | dotnet csharpier

Thanks go to @​jamesfoster for reporting the issue.

Full Changelog: belav/csharpier@0.28.1...0.28.2

v0.28.1

Compare Source

What's Changed
Third party .editorconfig leading to: Error Failure parsing editorconfig files #​1227

When CSharpier encountered an invalid .editorconfig file, it would throw an exception and not format files. These files could appear in 3rd party code (for example within node_modules). CSharpier now ignores invalid lines in .editorconfigs

Thanks go to @​K0Te for reporting the issue

Full Changelog: belav/csharpier@0.28.0...0.28.1

v0.28.0

Compare Source

What's Changed

Fix dedented method call if there is a long chain #​1154

In some cases of method chains, the first invocation would end up dedented.

// 0.27.3
o.Property.CallMethod(
    someParameter_____________________________,
    someParameter_____________________________
)
    .CallMethod()
    .CallMethod();

// 0.28.0
o.Property.CallMethod(
        someParameter_____________________________,
        someParameter_____________________________
    )
    .CallMethod()
    .CallMethod();
Extra newline in switch case statement with curly braces [#​1192][https://github.com/belav/csharpier/issues/1192](https://togithub.com/belav/csharpier/issues/1192)2

If a case statement started with a block it would get an extra new line

// 0.27.3
switch (someValue)
{
    case 0:
    {
        // dedented because the only statement is a block
        break;
    }

    case 1:

        {
            // indented because there are two statements, a block then a break
        }
        break;
}

// 0.28.0
// 0.27.3
switch (someValue)
{
    case 0:
    {
        // dedented because the only statement is a block
        break;
    }

    case 1:
        {
            // indented because there are two statements, a block then a break
        }
        break;
}

Thanks go to @​emberTrev for reporting the bug.

Handle more editorconfig glob patterns. #​1214

The editorconfig parsing was not handling glob patterns that contained braces.


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

Copy link

codecov bot commented Apr 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.27%. Comparing base (55ecbfd) to head (8375d75).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #114   +/-   ##
=======================================
  Coverage   97.27%   97.27%           
=======================================
  Files          26       26           
  Lines         954      954           
  Branches       50       50           
=======================================
  Hits          928      928           
  Misses         18       18           
  Partials        8        8           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@renovate renovate bot force-pushed the renovate/csharpier branch from af3acb0 to 4eb84fb Compare April 17, 2024 00:31
@renovate renovate bot changed the title Update CSharpier to v0.28.0 Update CSharpier to v0.28.1 Apr 17, 2024
@renovate renovate bot force-pushed the renovate/csharpier branch from 4eb84fb to ae41d0b Compare April 26, 2024 17:33
@renovate renovate bot changed the title Update CSharpier to v0.28.1 Update CSharpier to v0.28.2 Apr 26, 2024
@renovate renovate bot force-pushed the renovate/csharpier branch from ae41d0b to aa785e8 Compare May 1, 2024 17:54
@renovate renovate bot force-pushed the renovate/csharpier branch 2 times, most recently from 752c56b to 325f2b4 Compare May 1, 2024 18:04
@MadsBrun MadsBrun enabled auto-merge (squash) May 1, 2024 18:04
@renovate renovate bot force-pushed the renovate/csharpier branch 4 times, most recently from 5a61839 to cc9333d Compare May 1, 2024 18:23
@renovate renovate bot force-pushed the renovate/csharpier branch from cc9333d to 8375d75 Compare May 1, 2024 18:27
@MadsBrun MadsBrun merged commit b616608 into main May 1, 2024
8 checks passed
@MadsBrun MadsBrun deleted the renovate/csharpier branch May 1, 2024 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant