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

chore(deps): update dependency csharpier to v0.28.2 #73

Merged
merged 2 commits into from
May 18, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jan 24, 2024

Mend Renovate

This PR contains the following updates:

Package Type Update Change OpenSSF
csharpier nuget minor 0.27.0 -> 0.28.2 OpenSSF Scorecard

Release Notes

belav/csharpier (csharpier)

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.

v0.27.3

Compare Source

[*.cs]
indent_size = 4
tab_width = 4

v0.27.2

Compare Source

What's Changed

Orphan variable since 0.27.1 #​1153

0.27.1 introduced the following formatting regression, resulting in short variables being orphaned on a line

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

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

Thanks go to @​aurnoi1 for reporting the bug

Better support for CSharp Script #​1141

Version 0.27.1 parsed .csx files as if they were C#, so it could only format simple ones. It now parses them as CSharpScript files so it can format them properly.

Thanks go to @​Eptagone for reporting the bug.

Full Changelog: belav/csharpier@0.27.1...0.27.2

v0.27.1

Compare Source

What's Changed

Support for CSharp Script #​1141

Previously CSharpier would only format files matching *.cs which prevented it from formatting C# script files. It now formats *.{cs,csx}

Thanks go to @​Eptagone for the suggestion

Weird formatting of invocation chain #​1130

Invocation chains that started with an identifier <= 4 characters were causing a strange break in the first method call. There were other edge cases cleaned up while working on the fix.

// 0.27.0
var something________________________________________ = x.SomeProperty.CallMethod(
    longParameter_____________,
    longParameter_____________
)
    .CallMethod();

// 0.27.1
var something________________________________________ = x
    .SomeProperty.CallMethod(longParameter_____________, longParameter_____________)
    .CallMethod();
// 0.27.0
var someLongValue_________________ = memberAccessExpression[
    elementAccessExpression
].theMember______________________________();

// 0.27.1
var someLongValue_________________ = memberAccessExpression[elementAccessExpression]
    .theMember______________________________();
// 0.27.0
someThing_______________________
    ?.Property
    .CallMethod__________________()
    .CallMethod__________________();

// 0.27.1
someThing_______________________
    ?.Property.CallMethod__________________()
    .CallMethod__________________();

Thanks go to @​Rudomitori for reporting the issue

"Failed syntax tree validation" for raw string literals #​1129

When an interpolated raw string changed indentation due to CSharpier formatting, CSharpier was incorrectly reporting it as failing syntax tree validation.

// input
CallMethod(CallMethod(
   $$"""
   SomeString
   """, someValue));

// output
CallMethod(
    CallMethod(
        $$"""
        SomeString
        """,
        someValue
    )
);

Thanks go to @​Rudomitori for reporting the issue

Adding experimental support using HTTP for the extensions to communicate with CSharpier #​1137

The GRPC support added in 0.27.0 increased the size of the nuget package significantly and has been removed.

CSharpier can now start a kestrel web server to support communication with the extensions once they are all updated.

Full Changelog: belav/csharpier@0.27.0...0.27.1


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 this update 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.

@renovate renovate bot requested a review from JamieJamesJamie January 24, 2024 00:51
@renovate renovate bot force-pushed the renovate/csharpier-0.x branch from f7dbcd6 to 7e641b8 Compare January 28, 2024 19:36
@renovate renovate bot changed the title chore(deps): update dependency csharpier to v0.27.1 chore(deps): update dependency csharpier to v0.27.2 Jan 28, 2024
@renovate renovate bot force-pushed the renovate/csharpier-0.x branch from 7e641b8 to 39c612b Compare February 7, 2024 18:39
@renovate renovate bot force-pushed the renovate/csharpier-0.x branch from 39c612b to ddfc9d8 Compare February 18, 2024 19:09
@renovate renovate bot changed the title chore(deps): update dependency csharpier to v0.27.2 chore(deps): update dependency csharpier to v0.27.3 Feb 18, 2024
@renovate renovate bot force-pushed the renovate/csharpier-0.x branch 2 times, most recently from 99749f7 to 47c3bdb Compare March 29, 2024 16:36
@renovate renovate bot changed the title chore(deps): update dependency csharpier to v0.27.3 chore(deps): update dependency csharpier to v0.28.0 Apr 8, 2024
@renovate renovate bot force-pushed the renovate/csharpier-0.x branch from 47c3bdb to 1cc81b4 Compare April 8, 2024 01:15
@renovate renovate bot force-pushed the renovate/csharpier-0.x branch from 1cc81b4 to ee0dea5 Compare April 17, 2024 02:09
@renovate renovate bot changed the title chore(deps): update dependency csharpier to v0.28.0 chore(deps): update dependency csharpier to v0.28.1 Apr 17, 2024
@renovate renovate bot force-pushed the renovate/csharpier-0.x branch from ee0dea5 to 154f32d Compare April 26, 2024 17:41
@renovate renovate bot changed the title chore(deps): update dependency csharpier to v0.28.1 chore(deps): update dependency csharpier to v0.28.2 Apr 26, 2024
@renovate renovate bot force-pushed the renovate/csharpier-0.x branch from 154f32d to 0bc8f53 Compare May 18, 2024 21:10
@JamieJamesJamie JamieJamesJamie merged commit 487ca57 into main May 18, 2024
7 checks passed
@renovate renovate bot deleted the renovate/csharpier-0.x branch May 18, 2024 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant