Skip to content

Fix Copilot setup: switch to Windows runner, pin actions to full commit SHAs#206

Merged
markusrt merged 5 commits into
masterfrom
copilot/186-speziesidentifikation-maldi
Jun 10, 2026
Merged

Fix Copilot setup: switch to Windows runner, pin actions to full commit SHAs#206
markusrt merged 5 commits into
masterfrom
copilot/186-speziesidentifikation-maldi

Conversation

Copilot AI commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

The Copilot setup steps were running on ubuntu-latest with Mono, which is incompatible with this .NET Framework 4.8 solution. Additionally, all action references used mutable version tags instead of pinned commit SHAs.

Changes

copilot-setup-steps.yml

  • runs-on: ubuntu-latestwindows-latest — native MSBuild instead of Mono
  • Replaced Install Mono toolchain step with microsoft/setup-msbuild@v2
  • Added msbuild build step to pre-compile the solution and validate the environment
  • Pinned all three actions (actions/checkout, setup-msbuild, setup-nuget) to full commit SHAs

main.yml

  • Pinned all four actions to full commit SHAs for consistency (checkout, setup-msbuild, setup-nuget, Setup-VSTest)

Version tags are retained as inline comments (e.g. # v2) for readability:

- uses: microsoft/setup-msbuild@6fb02220983dee41ce7ae257b6f4d8f9bf5ed4ce # v2
- uses: nuget/setup-nuget@d105a947828025cd7a980103c35ba2bfae586d0f # v2

Copilot AI changed the title [WIP] Implement species identification for Nm and Hi using MALDI Switch Copilot setup steps to Windows runner with native MSBuild Jun 10, 2026
@markusrt markusrt marked this pull request as ready for review June 10, 2026 08:20
Copilot AI changed the title Switch Copilot setup steps to Windows runner with native MSBuild Fix Copilot setup: switch to Windows runner, pin actions to full commit SHAs Jun 10, 2026
@sonarqubecloud

Copy link
Copy Markdown

@markusrt markusrt merged commit 91b786c into master Jun 10, 2026
3 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.

2 participants