Skip to content

Add CSharpier formatting via CSharpier.MSBuild#33

Merged
Tyrrrz merged 3 commits intoprimefrom
copilot/add-csharpier-to-solution
Apr 15, 2026
Merged

Add CSharpier formatting via CSharpier.MSBuild#33
Tyrrrz merged 3 commits intoprimefrom
copilot/add-csharpier-to-solution

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 15, 2026

Integrates CSharpier into the solution for consistent code formatting, using the MSBuild package so formatting runs automatically as part of the build rather than requiring a separate tool invocation.

Changes

  • Directory.Packages.props — pins CSharpier.MsBuild 1.2.6
  • PowerKit/PowerKit.csproj — references CSharpier.MsBuild with PrivateAssets="all"
  • PowerKit.Tests/PowerKit.Tests.csproj — references CSharpier.MsBuild with PrivateAssets="all"
  • .config/dotnet-tools.json — removed; the MSBuild package supersedes the local tool manifest
  • C# source files — reformatted across PowerKit/ and PowerKit.Tests/ (45 files) to match CSharpier's style

Copilot AI and others added 2 commits April 15, 2026 09:23
@Tyrrrz
Copy link
Copy Markdown
Owner

Tyrrrz commented Apr 15, 2026

@copilot let's actually move each package refence to each csproj

@Tyrrrz Tyrrrz marked this pull request as ready for review April 15, 2026 09:38
Copilot AI review requested due to automatic review settings April 15, 2026 09:38
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 15, 2026

@copilot let's actually move each package refence to each csproj

Done in bc63de7 — removed the reference from Directory.Build.props and added <PackageReference Include="CSharpier.MsBuild" PrivateAssets="all" /> to both PowerKit/PowerKit.csproj and PowerKit.Tests/PowerKit.Tests.csproj.

Copilot AI requested a review from Tyrrrz April 15, 2026 09:41
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Integrates CSharpier into the build via CSharpier.MsBuild to enforce consistent C# formatting across the solution.

Changes:

  • Add centralized package version pin for CSharpier.MsBuild and reference it solution-wide via Directory.Build.props.
  • Apply CSharpier-driven formatting updates across library and test source files (primarily whitespace, wrapping, and using ordering).

Reviewed changes

Copilot reviewed 46 out of 47 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Directory.Packages.props Pins CSharpier.MsBuild version via central package management.
Directory.Build.props Adds solution-wide PackageReference to CSharpier.MsBuild with PrivateAssets="all".
PowerKit/TempFile.cs Formatting only (wrapping long call; using ordering).
PowerKit/TempDirectory.cs using ordering formatting.
PowerKit/LockFile.cs using ordering formatting.
PowerKit/Extensions/TimeSpanExtensions.cs using ordering formatting.
PowerKit/Extensions/TextReaderExtensions.cs using ordering + formatting of while condition.
PowerKit/Extensions/StringExtensions.cs using ordering formatting.
PowerKit/Extensions/StringBuilderExtensions.cs using ordering formatting.
PowerKit/Extensions/StreamExtensions.cs using ordering formatting.
PowerKit/Extensions/PathExtensions.cs Formatting only (array layout, wrapping long expressions).
PowerKit/Extensions/LongExtensions.cs using ordering formatting.
PowerKit/Extensions/IntExtensions.cs using ordering formatting.
PowerKit/Extensions/FileExtensions.cs using ordering + wrapping FileStream ctor args.
PowerKit/Extensions/ExceptionExtensions.cs Removes trailing whitespace line.
PowerKit/Extensions/EnvironmentExtensions.cs Wraps long Environment.* calls for formatting consistency.
PowerKit/Extensions/EnumerableExtensions.cs using ordering formatting.
PowerKit/Extensions/EncodingExtensions.cs Formatting only (wrap long method signatures/ternary).
PowerKit/Extensions/DoubleExtensions.cs using ordering formatting.
PowerKit/Extensions/DirectoryExtensions.cs using ordering formatting.
PowerKit/Extensions/DecimalExtensions.cs using ordering formatting.
PowerKit/Extensions/DateTimeOffsetExtensions.cs Formatting only (expression-bodied method wrapping).
PowerKit/Extensions/DateTimeExtensions.cs using ordering formatting.
PowerKit/Extensions/CultureInfoExtensions.cs using ordering formatting.
PowerKit/Extensions/ConsoleExtensions.cs Wraps method parameters for formatting consistency.
PowerKit/Extensions/ComparableExtensions.cs Formatting only (generic constraint wrapping).
PowerKit/Extensions/CollectionExtensions.cs using ordering formatting.
PowerKit/Extensions/BinaryWriterExtensions.cs using ordering formatting.
PowerKit/Extensions/BinaryReaderExtensions.cs using ordering + formatting of conditional expression property.
PowerKit/Extensions/AsyncEnumerableExtensions.cs Formatting only (wrap while / await foreach lines).
PowerKit/Extensions/AssemblyExtensions.cs Formatting only (wrap null-coalescing chain).
PowerKit/Extensions/ArrayPoolExtensions.cs using ordering formatting.
PowerKit.Tests/StringBuilderExtensionsTests.cs Formatting only (method chaining wrapping).
PowerKit.Tests/StreamExtensionsTests.cs Removes trailing whitespace line.
PowerKit.Tests/ProcessExtensionsTests.cs Formatting only (wrap Process.Start args).
PowerKit.Tests/PowerKit.Tests.csproj Removes stray blank lines / whitespace formatting.
PowerKit.Tests/GuidExtensionsTests.cs Formatting only (method chaining wrapping).
PowerKit.Tests/FileExtensionsTests.cs Formatting only (wrap long File.SetAttributes calls).
PowerKit.Tests/EnumerableExtensionsTests.cs Formatting only (wrap assertions / chaining).
PowerKit.Tests/DirectoryExtensionsTests.cs Formatting only (wrap long call).
PowerKit.Tests/DateTimeOffsetExtensionsTests.cs Formatting only (wrap chaining).
PowerKit.Tests/CultureInfoExtensionsTests.cs Formatting only (wrap chaining / line breaks).
PowerKit.Tests/CollectionExtensionsTests.cs Formatting only (cast spacing).
PowerKit.Tests/BinaryReaderExtensionsTests.cs Removes trailing whitespace line.
PowerKit.Tests/AsyncEnumerableExtensionsTests.cs Formatting only (wrap assertions).
PowerKit.Tests/AssemblyExtensionsTests.cs Formatting only (wrap assertion chaining).
PowerKit.Tests/AggregateExceptionExtensionsTests.cs Formatting only (wrap chaining / ctor nesting).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants