-
Notifications
You must be signed in to change notification settings - Fork 1
Ensure .NET 8, 9, and 10 SDKs are available in CI workflow #132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: geraldbarre-workleap <240971877+geraldbarre-workleap@users.noreply.github.com>
|
@copilot .github/workflows/publish.yml:35 the publish workflow doesn't run the tests, so it only needs the framework from the global.json file. .github/workflows/ci.yml:32 You can simplify that. |
Co-authored-by: geraldbarre-workleap <240971877+geraldbarre-workleap@users.noreply.github.com>
Updated in commit 0daea59:
|
Co-authored-by: geraldbarre-workleap <240971877+geraldbarre-workleap@users.noreply.github.com>
|
@copilot Can you also update the SDK in the global.json file to 10.0.101, and update the setup .NET step to install 9 instead of 10 |
…of 10 Co-authored-by: geraldbarre-workleap <240971877+geraldbarre-workleap@users.noreply.github.com>
Updated in commit 23f8836:
|
There was a problem hiding this 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 attempts to update the CI workflow to ensure .NET 8, 9, and 10 SDKs are available for testing, but there appears to be a significant mismatch between the PR's stated purpose and the actual project requirements.
Key concerns:
- The project only targets net8.0, net462, and netstandard2.0 frameworks - there are no .NET 9 or .NET 10 target frameworks in the codebase
- The global.json update changes the required SDK from 9.0.306 to 10.0.101 without clear justification
- Installing multiple SDK versions (8, 9, and 10) when only .NET 8 target frameworks exist adds unnecessary CI overhead
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| global.json | Updates SDK version to 10.0.101, despite the project not targeting any .NET 10 frameworks |
| .github/workflows/ci.yml | Adds .NET setup step to install SDKs for versions 8, 9, and 10, though only .NET 8 target frameworks exist in the project |
CI workflow needs explicit .NET 8, 9, and 10 SDK setup to support multi-version testing requirements.
Changes
global-json-file: "global.json"(for .NET 10.0.101) plus additional versions 9.0.x and 8.0.x via multi-linedotnet-versionparameterglobal-json-file: "global.json"as the publish workflow doesn't run tests)This ensures all required SDK versions (8, 9, and 10) are available in the CI workflow for testing the library's multi-target framework build (
net462,net8.0,netstandard2.0), while keeping the publish workflow simple with only the version from global.json.Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.