ci: move from dotnet-format tool to .NET SDK command (#4918)

* ci: remove installing global tool dotnet-format

* ci: Remove AttributeReaderTests.cs exclusion from dotnet format

The exclusion was added as a workaround because dotnet-format was
incorrectly deleting assertions in that file. This issue has since
been resolved so the exclusion is no longer necessary.

Co-Authored-By: Claude <noreply@anthropic.com>

* Format code

* Revert "Format code"

This reverts commit bba983f.

* Revert "ci: Remove AttributeReaderTests.cs exclusion from dotnet format"

This reverts commit bbcefcb.

* ci: Add JSON format report and upload as artifact

Generate a dotnet format report via --report and upload it as a CI
artifact so formatting issues can be inspected without re-running the
job locally.

Refs #4918
Co-Authored-By: Claude <noreply@anthropic.com>

* ci: Move --report before --exclude arguments

Refs #4918
Co-Authored-By: Claude <noreply@anthropic.com>

* ci: Consolidate --exclude arguments into a single flag

Refs #4918
Co-Authored-By: Claude <noreply@anthropic.com>

* ci: Exclude format report directory from git tracking

Write a .gitignore into the report directory after uploading the
artifact so the generated JSON report is not picked up as a git
change by the subsequent commit step.

Refs #4918
Co-Authored-By: Claude <noreply@anthropic.com>

* ci: Only write .gitignore if report directory exists

Avoid unconditionally creating the report directory — only write the
.gitignore when dotnet format actually produced the directory.

Refs #4918
Co-Authored-By: Claude <noreply@anthropic.com>

* ci: use actions/upload-artifact@v7 direct file upload (unzipped)

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Sentry Github Bot <bot+github-bot@sentry.io>

Author: 3 people

.github/workflows/format-code.yml

@@ -28,17 +28,28 @@ jobs:
           dotnet workload restore
           dotnet restore Sentry.slnx --nologo
 
-      - name: Install dotnet format
-        run: dotnet tool install -g dotnet-format
-
       - name: Format Code
         # We exclude `./**/*OptionsSetup.cs` from the format because the tool struggles with
         # source generators.
         #
         # We exclude `test/Sentry.Tests/AttributeReaderTests.cs` because dotnet-format tries to
         # delete assertions in there.
         # - see https://github.com/getsentry/sentry-dotnet/pull/4911#discussion_r2795717887
-        run: dotnet format Sentry.slnx --no-restore --exclude ./modules --exclude ./**/*OptionsSetup.cs --exclude ./test/Sentry.Tests/AttributeReaderTests.cs
+        run: dotnet format Sentry.slnx --no-restore --report ./dotnet-format-report --exclude ./modules ./**/*OptionsSetup.cs ./test/Sentry.Tests/AttributeReaderTests.cs
+
+      - name: Upload Format Report
+        if: ${{ always() }}
+        uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # v7.0.0
+        with:
+          archive: false
+          path: ./dotnet-format-report/
+          if-no-files-found: ignore
+
+      - name: Exclude Format Report from Git
+        run: |
+          if [ -d ./dotnet-format-report ]; then
+            echo '*' > ./dotnet-format-report/.gitignore
+          fi
 
       # actions/checkout fetches only a single commit in a detached HEAD state. Therefore
       # we need to pass the current branch, otherwise we can't commit the changes.