Skip to content

Update NuGet publish workflow for current component set and .NET 10 support #151

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.

Sign up for GitHub

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

Merged
JusterZhu merged 5 commits into from
Feb 12, 2026
Merged

Update NuGet publish workflow for current component set and .NET 10 support #151

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
ded1964
Initial plan
Copilot Feb 12, 2026
1b14604
Update workflow to include correct NuGet components
Copilot Feb 12, 2026
01c9d12
Fix YAML formatting issues
Copilot Feb 12, 2026
591c8bd
Add GeneralUpdate.Common back to the component list
Copilot Feb 12, 2026
289aaf3
Update workflow to use .NET 10 for Drivelution support
Copilot Feb 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions .github/workflows/dotnet-ci.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
- name: 安装.NET SDK
uses: actions/setup-dotnet@v4
with:
dotnet-version: '9.0.x' # 与现有CI配置保持一致
dotnet-version: '10.0.x' # 支持GeneralUpdate.Drivelution的.NET 10目标框架

- name: 恢复依赖
run: dotnet restore ./src/c#/GeneralUpdate.sln # 使用解决方案统一恢复
Expand All @@ -33,9 +33,11 @@ jobs:
"GeneralUpdate.ClientCore",
"GeneralUpdate.Common",
Copy link
Collaborator

@JusterZhu JusterZhu Feb 12, 2026

Choose a reason for hiding this comment

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

GeneralUpdate.Common需要加上

Copy link
Collaborator

@JusterZhu JusterZhu Feb 12, 2026

Choose a reason for hiding this comment

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

@copilot check for this

Copy link
Contributor Author

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

Added GeneralUpdate.Common back to the component list in commit 591c8bd.

"GeneralUpdate.Core",
"GeneralUpdate.Differential"
"GeneralUpdate.Differential",
"GeneralUpdate.Drivelution",
"GeneralUpdate.Extension"
)

foreach ($project in $projects) {
dotnet pack ./src/c#/$project/$project.csproj `
Comment on lines +38 to 42
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

shell: pwsh means # starts a comment in PowerShell. The paths ./src/c#/... will be parsed as ./src/c with the remainder treated as a comment, which will break dotnet pack (and similarly the earlier restore/build steps). Quote or escape the c# paths (e.g., wrap in quotes or use an escaped #) so PowerShell passes the correct file path to dotnet.

Copilot uses AI. Check for mistakes.
-c Release `
Expand All @@ -53,4 +55,4 @@ jobs:
name: Release v${{ github.event.inputs.version }}
files: ./nupkgs/*.nupkg
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}