Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 17, 2025

Upgrades the CLI tool runtime from .NET 8 to .NET 10 and ensures GitHub Actions workflows support the new version.

Changes

Core Projects

  • CLI and all test projects: net8.0net10.0 target framework
  • global.json: SDK version 6.0.42810.0.100

CI/CD Workflows
Added 10.0.x to .NET setup steps in:

  • unit-tests.yml, smoke-tests.yml
  • regression-tests.yml (3 jobs: C#, JMeter, TypeScript)
  • production-tests.yml (3 jobs: C#, JMeter, TypeScript)
  • vscode.yml, release.yml

Documentation

  • README.md, docs/CLI.md: Updated runtime requirement to .NET 10.0

Backward Compatibility

Workflows maintain support for .NET 2.1.x through 10.0.x. Only the CLI tool itself requires .NET 10 runtime.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • collector.exceptionless.io
    • Triggering command: refitter --settings-file /home/REDACTED/work/apiclientcodegen/apiclientcodegen/src/Swagger.refitter --simple-output (dns block)
  • heartbeat.exceptionless.io
    • Triggering command: refitter --settings-file /home/REDACTED/work/apiclientcodegen/apiclientcodegen/src/Swagger.refitter --simple-output (dns block)
  • petstore.swagger.io
    • Triggering command: /usr/share/dotnet/dotnet exec --runtimeconfig /home/REDACTED/work/apiclientcodegen/apiclientcodegen/src/Core/ApiClientCodeGen.Core.Tests/bin/Debug/net10.0/ApiClientCodeGen.Core.Tests.runtimeconfig.json --depsfile /home/REDACTED/work/apiclientcodegen/apiclientcodegen/src/Core/ApiClientCodeGen.Core.Tests/bin/Debug/net10.0/ApiClientCodeGen.Core.Tests.deps.json /home/REDACTED/work/apiclientcodegen/apiclientcodegen/src/Core/ApiClientCodeGen.Core.Tests/bin/Debug/net10.0/testhost.dll --port 32959 --endpoint 127.0.0.1:032959 --role client --parentprocessid 4315 --telemetryoptedin false (dns block)
  • westeurope-5.in.applicationinsights.azure.com
    • Triggering command: refitter --settings-file /home/REDACTED/work/apiclientcodegen/apiclientcodegen/src/Swagger.refitter --simple-output (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Upgrade the CLI tool to run on .NET 10. Ensure that the Github workflows are all compatible with .NET 10. Update the software requirements in the README files to indicate that .NET 10 is needed to run the tool


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@coderabbitai
Copy link

coderabbitai bot commented Nov 17, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI and others added 3 commits November 17, 2025 13:23
Co-authored-by: christianhelle <710400+christianhelle@users.noreply.github.com>
Co-authored-by: christianhelle <710400+christianhelle@users.noreply.github.com>
Co-authored-by: christianhelle <710400+christianhelle@users.noreply.github.com>
Copilot AI changed the title [WIP] Upgrade CLI tool and workflows to .NET 10 Upgrade CLI tool to .NET 10 Nov 17, 2025
Copilot AI requested a review from christianhelle November 17, 2025 13:27
Copilot finished work on behalf of christianhelle November 17, 2025 13:27
@sonarqubecloud
Copy link

@christianhelle christianhelle marked this pull request as ready for review November 18, 2025 08:22
Copilot AI review requested due to automatic review settings November 18, 2025 08:22
Copilot finished reviewing on behalf of christianhelle November 18, 2025 08:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR upgrades the CLI tool and related test projects from .NET 8.0 to .NET 10.0, updating the SDK version in global.json from 6.0.428 to 10.0.100, and ensuring GitHub Actions workflows support the new runtime version.

Key Changes:

  • Target framework migration from net8.0 to net10.0 for CLI tool and all test projects
  • SDK version bump to 10.0.100 with rollForward policy change from "latestMajor" to "latestMinor" for more controlled version management
  • GitHub Actions workflows updated to install .NET 10.0.x runtime alongside existing versions (2.1.x through 9.0.x)

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/global.json Updated SDK version to 10.0.100 and changed rollForward policy to latestMinor
src/Core/ApiClientCodeGen.Core.Tests/ApiClientCodeGen.Core.Tests.csproj Updated target framework from net8.0 to net10.0
src/Core/ApiClientCodeGen.Core.IntegrationTests/ApiClientCodeGen.Core.IntegrationTests.csproj Updated target framework from net8.0 to net10.0
src/CLI/ApiClientCodeGen.CLI/ApiClientCodeGen.CLI.csproj Updated target framework from net8.0 to net10.0
src/CLI/ApiClientCodeGen.CLI.Tests/ApiClientCodeGen.CLI.Tests.csproj Updated target framework from net8.0 to net10.0
docs/CLI.md Updated runtime requirement documentation from .NET 6.0 to .NET 10.0
README.md Updated runtime requirement documentation from .NET 6.0 to .NET 10.0
.github/workflows/vsix.yml Added .NET 10.0.x setup step
.github/workflows/vscode.yml Updated .NET version from 6.0.x to 10.0.x
.github/workflows/unit-tests.yml Added 9.0.x and 10.0.x to .NET setup, updated test commands to use net10.0 framework
.github/workflows/sonar-cloud.yml Added duplicate .NET 10.0.x setup steps (redundant configuration)
.github/workflows/smoke-tests.yml Added 10.0.x to .NET setup in all job configurations
.github/workflows/release.yml Added/updated .NET setup steps to use 10.0.x
.github/workflows/regression-tests.yml Added 10.0.x to .NET setup across all test jobs
.github/workflows/production-tests.yml Added 10.0.x to .NET setup across all test jobs
.github/workflows/cli-tool.yml Added .NET 10.0.x setup step

Comment on lines +57 to +60
- name: Setup .NET
uses: actions/setup-dotnet@v5
with:
dotnet-version: '10.0.x'
Copy link

Copilot AI Nov 18, 2025

Choose a reason for hiding this comment

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

Duplicate .NET setup step. This step is identical to the earlier setup at line 27-30 and can be removed to avoid redundancy.

Suggested change
- name: Setup .NET
uses: actions/setup-dotnet@v5
with:
dotnet-version: '10.0.x'

Copilot uses AI. Check for mistakes.
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