Skip to content

build: add csharpier tooling#21

Merged
mnbuhl merged 1 commit intomainfrom
setup-csharpier
May 5, 2026
Merged

build: add csharpier tooling#21
mnbuhl merged 1 commit intomainfrom
setup-csharpier

Conversation

@mnbuhl
Copy link
Copy Markdown
Owner

@mnbuhl mnbuhl commented May 5, 2026

Summary

  • Pin CSharpier as a local .NET tool so contributors use the same formatter version.
  • Add a CSharpier ignore file for EF Core generated migration output.
  • Add CI coverage so formatting is checked in pull requests.

Changes

  • Added .config/dotnet-tools.json with CSharpier 1.2.6.
  • Added .csharpierignore for **/Migrations/**.
  • Updated the verify workflow to restore .NET tools and run dotnet csharpier check ..
  • Applied the current CSharpier formatting output to two existing C# files.

Risk and rollout

  • Low risk: this is tooling and formatting only, with no intended runtime behavior change.
  • CI now fails on unformatted C# changes, so formatter drift is caught before merge.
  • No Git hooks are added or required.

Notes for reviewers

  • The formatting gate lives in CI; local formatting remains an explicit dotnet csharpier format . command.

@mnbuhl mnbuhl marked this pull request as ready for review May 5, 2026 15:17
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 53f9be7432

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .githooks/pre-commit Outdated
@mnbuhl mnbuhl force-pushed the setup-csharpier branch from 53f9be7 to 5584894 Compare May 5, 2026 15:20
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

Code Coverage

Package Line Rate Health
Atomizer 72%
Atomizer 72%
Atomizer 14%
Atomizer.EntityFrameworkCore 90%
Atomizer 14%
Atomizer.EntityFrameworkCore 90%
Atomizer 14%
Atomizer.EntityFrameworkCore 90%
Atomizer 72%
Summary 60% (7347 / 13050)

@mnbuhl mnbuhl merged commit 968046f into main May 5, 2026
1 check passed
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