Skip to content

Run dotnet format whitespace so the pre-commit check passes - #173

Merged
theryansmee merged 1 commit into
feature/greenfieldfrom
fix/dotnet-format-whitespace
Oct 5, 2026
Merged

theryansmee merged 1 commit into
feature/greenfieldfrom
fix/dotnet-format-whitespace

Conversation

@theryansmee

Copy link
Copy Markdown

The pre-commit check (dotnet format whitespace, .NET 8, same as CI) was failing on greenfield for every PR, including docs-only ones.

This runs the same hook over the repo. It changes whitespace only in 252 C# files: brace placement and indentation. Verified by comparing every changed file with all whitespace stripped, before and after, and they match. No line-ending changes, and no non-C# files touched.

Heads-up: draft #168 edits BuilderWindow.cs and SqEditorAppApi.cs, which this also reformats, so it will need a rebase. Running dotnet format whitespace --folder on that branch before rebasing resolves it.

@coderabbitai

coderabbitai Bot commented Oct 5, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: e1e3ba62-3526-472c-aa44-7ad8ded2689c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@theryansmee
theryansmee merged commit 0741bea into feature/greenfield Oct 5, 2026
2 checks 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