Skip to content

[WIP] Update .NET build and test steps to avoid duplicate compilation#988

Merged
dgee2 merged 3 commits into
mainfrom
codex/align-dotnet-build-and-test-steps
Apr 15, 2026
Merged

[WIP] Update .NET build and test steps to avoid duplicate compilation#988
dgee2 merged 3 commits into
mainfrom
codex/align-dotnet-build-and-test-steps

Conversation

@Codex

@Codex Codex AI commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>Phase 1: Align .NET build and test steps to avoid duplicate compilation</issue_title>
<issue_description>## Summary
Update the GitHub Actions workflow so .NET is restored once, built once in Release, and tested with --no-build.

Change

Replace the current backend sequence with:

  • dotnet restore
  • dotnet build --configuration Release --no-restore
  • dotnet test --configuration Release --no-build

Why

Recent analysis indicates the workflow likely rebuilds because build and test configurations are misaligned. This is the lowest-risk first improvement.

Acceptance criteria

  • Backend workflow uses a single Release build
  • Tests run with --no-build
  • CI remains green
  • Timing before and after is captured for comparison</issue_description>

Comments on the Issue (you are @codex[agent] in this section)

Co-authored-by: dgee2 <5671841+dgee2@users.noreply.github.com>
@Codex Codex AI requested a review from dgee2 April 15, 2026 21:27
@dgee2 dgee2 marked this pull request as ready for review April 15, 2026 21:27
Copilot AI review requested due to automatic review settings April 15, 2026 21:27
@dgee2 dgee2 enabled auto-merge (squash) April 15, 2026 21:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the GitHub Actions workflow’s .NET build/test sequence to align configurations and prevent dotnet test from triggering an additional compilation, improving CI efficiency for the backend portion of the pipeline.

Changes:

  • Build step now uses dotnet build --configuration Release --no-restore.
  • Test step now uses dotnet test --configuration Release --no-build.

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

Comment thread .github/workflows/main.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@sonarqubecloud

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


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

@dgee2 dgee2 merged commit e10e1c9 into main Apr 15, 2026
14 checks passed
@dgee2 dgee2 deleted the codex/align-dotnet-build-and-test-steps branch April 15, 2026 21:38
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.

Phase 1: Align .NET build and test steps to avoid duplicate compilation

3 participants