Skip to content

chore: Separate restore from dotnet format for better error visibility#165

Merged
goloroden merged 1 commit intomainfrom
chore/separate-restore-from-format
Mar 10, 2026
Merged

chore: Separate restore from dotnet format for better error visibility#165
goloroden merged 1 commit intomainfrom
chore/separate-restore-from-format

Conversation

@goloroden
Copy link
Copy Markdown
Member

Summary

  • Add explicit restore target to Makefile that runs dotnet restore for both projects
  • Add --no-restore flag to all dotnet format calls in analyze and format targets
  • Make qa depend on restore as first step, and format depend on restore for standalone use

This ensures restore errors are surfaced directly instead of being swallowed by dotnet format's internal restore, which only throws System.Exception: Restore operation failed without details.

Test plan

  • make restore runs without errors
  • make qa runs without errors
  • make format runs without errors
  • CI passes on this PR

🤖 Generated with Claude Code

Run dotnet restore explicitly before dotnet format to surface restore
errors directly, as dotnet format swallows error details internally.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@goloroden goloroden requested a review from a team as a code owner March 10, 2026 08:33
@goloroden goloroden self-assigned this Mar 10, 2026
@goloroden goloroden merged commit 5593533 into main Mar 10, 2026
4 checks passed
@goloroden goloroden deleted the chore/separate-restore-from-format branch March 10, 2026 08:35
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