Skip to content

Fixing bug with range ignore + regions #1112

Fixing bug with range ignore + regions

Fixing bug with range ignore + regions #1112

Triggered via pull request March 24, 2024 18:05
@belavbelav
synchronize #1216
regions
Status Failure
Total duration 1m 42s
Artifacts

validate_pull_request.yml

on: pull_request
Run Tests
1m 5s
Run Tests
Check Formatting
13s
Check Formatting
Test CSharpier.MSBuild
1m 30s
Test CSharpier.MSBuild
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 6 warnings
Directives_Regions: Src/CSharpier.Tests/CSharpier.Tests.Generators/CSharpier.Tests.Generators.FormattingTestsGenerator/FormattingTests.cs#L272
Expected normalizedCode to be "public class ClassName { #region Region public int LongUglyMethod() => 42; #endregion } public class ClassName { #region Region public int LongUglyMethod() => 42; #endregion } #region class public class ClassName { #region method public void Method() { #region content #endregion } #endregion } #endregion public class ClassName { #region Region private int x; // trailing comment here shouldn't give extra indent to the endregion after it #endregion private int y; string RegionsIndentAndNewLineProperly = #region one @"using System;" #endregion one ; } class ClassName { #region Region // csharpier-ignore-start public string Field; // csharpier-ignore-end #endregion } " with a length of 813, but "public class ClassName { #region Region public int LongUglyMethod() => 42; #endregion } public class ClassName { #region Region public int LongUglyMethod() => 42; #endregion } #region class public class ClassName { #region method public void Method() { #region content #endregion } #endregion } #endregion public class ClassName { #region Region private int x; // trailing comment here shouldn't give extra indent to the endregion after it #endregion private int y; string RegionsIndentAndNewLineProperly = #region one @"using System;" #endregion one ; } class ClassName { #region Region // csharpier-ignore-start public string Field; // csharpier-ignore-end #endregion } " has a length of 809, differs near " " (index 687).
Run Tests
Process completed with exit code 1.
Check Formatting
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-dotnet@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Check Formatting
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Run Tests
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-dotnet@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Run Tests
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Test CSharpier.MSBuild
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-dotnet@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Test CSharpier.MSBuild
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/