Skip to content

Conversation

@pardeike
Copy link
Owner

Summary

  • ensure the Windows test setup action installs the .NET 10 SDK so global.json requirements are met

Testing

  • dotnet test HarmonyTests/HarmonyTests.csproj -c Release -f net9.0

Codex Task

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 18 to 22
- name: Setup .NET SDK ${{inputs.architecture}}
uses: actions/setup-dotnet@v4
with:
dotnet-version: '10.0.x'
architecture: ${{inputs.architecture}}

Choose a reason for hiding this comment

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

P1 Badge Install runtimes where setup-dotnet expects them

The new actions/setup-dotnet@v4 step installs the 10.0 SDK into the hosted tool cache and sets DOTNET_ROOT/disables multi-level lookup for the job. The later PowerShell block still drops all required runtimes into ${{steps.get-program-files.outputs.path}}/dotnet (Program Files). Because the active dotnet on PATH now points to the tool cache, it no longer sees those runtimes, so Windows .NET tests targeting netcoreapp3.x–net9.0 will fail with “No compatible framework version was found”. Install the extra runtimes into DOTNET_ROOT (or re-enable multi-level lookup) so the SDK can locate them.

Useful? React with 👍 / 👎.

@github-actions
Copy link

github-actions bot commented Nov 12, 2025

Test Results

   128 files     128 suites   7m 59s ⏱️
   280 tests    277 ✅   3 💤 0 ❌
35 568 runs  34 992 ✅ 576 💤 0 ❌

Results for commit 3559f50.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants