Update CI/CD for .NET 10, docs, and release automation by Chris-Wolfgang · Pull Request #41 · Chris-Wolfgang/ETL-Abstractions

Chris-Wolfgang · 2025-12-27T17:28:12Z

Add .NET 10.0.x support to all workflows and targets
Split build/test steps for Linux/macOS vs Windows for better cross-platform support
Run tests on net8.0 only for Linux/macOS; all frameworks on Windows
Enhance NuGet packaging: build/publish .nupkg and .snupkg, upload as artifacts
Automate GitHub release with attached NuGet packages and release notes
Update permissions and run publish job on Windows for release creation
Refactor docfx.json: clarify structure, exclude private members, expand content/resource globs, add modern template
Add verification of DocFX build output in workflow
Update toc.yml to include Articles section for improved navigation

Description

Fixes/Complete # (issue)

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Test A
Test B

Checklist

My code follows the style guidelines of this project
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes

Screenshots (if applicable)

Additional context

- Add .NET 10.0.x support to all workflows and targets - Split build/test steps for Linux/macOS vs Windows for better cross-platform support - Run tests on net8.0 only for Linux/macOS; all frameworks on Windows - Enhance NuGet packaging: build/publish .nupkg and .snupkg, upload as artifacts - Automate GitHub release with attached NuGet packages and release notes - Update permissions and run publish job on Windows for release creation - Refactor docfx.json: clarify structure, exclude private members, expand content/resource globs, add modern template - Add verification of DocFX build output in workflow - Update toc.yml to include Articles section for improved navigation

Copilot

Pull request overview

This pull request adds .NET 10.0 support across CI/CD workflows, enhances NuGet packaging with symbol packages, automates GitHub releases, and refactors DocFX documentation configuration with new navigation structure.

Key changes:

Add .NET 10.0 target framework support to all workflows (PR, release, and DocFX)
Split build/test steps between Linux/macOS (cross-platform targets only) and Windows (all targets including .NET Framework) for better cross-platform support
Enhance NuGet packaging to create and publish both .nupkg and .snupkg symbol packages with automated GitHub release creation
Refactor docfx.json configuration with expanded content patterns for articles, modern template, and changed output directory property
Add Articles section to documentation navigation

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 13 comments.

Show a summary per file

File	Description
.github/workflows/release.yaml	Adds .NET 10.0 support, splits build/test for cross-platform compatibility, enhances NuGet packaging with symbols, and automates GitHub release creation
.github/workflows/pr.yaml	Updates to include .NET 10.0 in the test matrix and adjusts framework targets for Linux/macOS builds
.github/workflows/docfx.yaml	Adds verification step to check DocFX build output exists before deployment
docfx_project/docfx.json	Refactors configuration with explicit project paths, adds articles content patterns, changes output property, adds modern template, and removes globalMetadata
docfx_project/toc.yml	Adds Articles navigation section and removes API homepage property

Comments suppressed due to low confidence (1)

.github/workflows/pr.yaml:53

The workflow attempts to build for 'net10.0' framework, but the main library project file (Wolfgang.Etl.Abstractions.csproj) only includes target frameworks up to net9.0. The TargetFrameworks property needs to be updated to include net10.0, otherwise this build will fail with 'The framework 'net10.0' is not supported by this project'.

          for fw in netstandard2.0 net8.0 net10.0; do
            echo "Building Wolfgang.Etl.Abstractions for $fw"
            dotnet build src/Wolfgang.Etl.Abstractions/Wolfgang.Etl.Abstractions.csproj \
              --no-restore \
              --configuration Release \
              --framework "$fw"
          done

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-27T17:32:17Z

+      "default",
+      "modern"


The 'modern' template may not be available in the default DocFX installation. DocFX typically ships with the 'default' and 'statictoc' templates. If 'modern' is a custom template, it needs to be present in the docfx_project directory. If this template doesn't exist, the build will fail. Consider removing this unless you've verified the template is available or will be added separately.

Suggested change

"default",

"modern"

"default"

Copilot · 2025-12-27T17:32:18Z

+          dotnet test tests/Wolfgang.Etl.Abstractions.Tests/Wolfgang.Etl.Abstractions.Tests.Unit.csproj \
+            --configuration Release \
+            --framework net8.0 \
+            --no-build \
+            --logger "trx" \
+            --results-directory "./TestResults"


The test project path is incorrect. It should be 'tests/Wolfgang.Etl.Abstractions.Tests.Unit/Wolfgang.Etl.Abstractions.Tests.Unit.csproj' instead of 'tests/Wolfgang.Etl.Abstractions.Tests/Wolfgang.Etl.Abstractions.Tests.Unit.csproj'. This will cause the test run to fail with 'project file does not exist'.

Copilot · 2025-12-27T17:32:18Z

+          dotnet test tests/Wolfgang.Etl.Abstractions.Tests/Wolfgang.Etl.Abstractions.Tests.Unit.csproj `
+            --configuration Release `
+            --no-build `
+            --logger "trx" `
+            --results-directory "./TestResults"


The test project path is incorrect. It should be 'tests/Wolfgang.Etl.Abstractions.Tests.Unit/Wolfgang.Etl.Abstractions.Tests.Unit.csproj' instead of 'tests/Wolfgang.Etl.Abstractions.Tests/Wolfgang.Etl.Abstractions.Tests.Unit.csproj'. This will cause the test run to fail with 'project file does not exist'.

@copilot open a new pull request to apply changes based on this feedback

Copilot · 2025-12-27T17:32:19Z

+            dotnet pack src/Wolfgang.Etl.Abstractions/Wolfgang.Etl.Abstractions.csproj 
+            --no-build `
+            --configuration Release `
+            --output ./nuget-packages `
+            --include-symbols `
+            --include-source `
+            -p:SymbolPackageFormat=snupkg


The PowerShell command is improperly formatted with the opening line containing the command name without a line continuation character. This will cause a syntax error. The opening 'dotnet pack' line should end with a backtick (`) to continue to the next line, or all arguments should be on the same line.

@copilot open a new pull request to apply changes based on this feedback

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot · 2025-12-27T17:58:22Z

@Chris-Wolfgang I've opened a new pull request, #42, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2025-12-27T17:59:05Z

@Chris-Wolfgang I've opened a new pull request, #43, to work on those changes. Once the pull request is ready, I'll request review from you.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Copilot · 2025-12-27T18:01:12Z

@Chris-Wolfgang I've opened a new pull request, #44, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2025-12-27T18:01:43Z

@Chris-Wolfgang I've opened a new pull request, #45, to work on those changes. Once the pull request is ready, I'll request review from you.

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot · 2025-12-27T18:03:11Z

@Chris-Wolfgang I've opened a new pull request, #46, to work on those changes. Once the pull request is ready, I'll request review from you.

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 11 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-27T18:08:02Z

+        if: runner.os == 'Windows'
+        shell: pwsh
+        run: |
+          dotnet test tests/Wolfgang.Etl.Abstractions.Tests/Wolfgang.Etl.Abstractions.Tests.Unit.csproj `


The test project path is incorrect. The actual test project is located at 'tests/Wolfgang.Etl.Abstractions.Tests.Unit/Wolfgang.Etl.Abstractions.Tests.Unit.csproj' but this references 'tests/Wolfgang.Etl.Abstractions.Tests/Wolfgang.Etl.Abstractions.Tests.Unit.csproj'. The directory name should be 'Wolfgang.Etl.Abstractions.Tests.Unit' not 'Wolfgang.Etl.Abstractions.Tests'.

Copilot · 2025-12-27T18:08:03Z

-            echo "Running tests for $proj"
-            dotnet test "$proj" --no-build --configuration Release --logger "trx" --results-directory "./TestResults"
-          done
+          dotnet test tests/Wolfgang.Etl.Abstractions.Tests/Wolfgang.Etl.Abstractions.Tests.Unit.csproj \


The test project path is incorrect. The actual test project is located at 'tests/Wolfgang.Etl.Abstractions.Tests.Unit/Wolfgang.Etl.Abstractions.Tests.Unit.csproj' but this references 'tests/Wolfgang.Etl.Abstractions.Tests/Wolfgang.Etl.Abstractions.Tests.Unit.csproj'. The directory name should be 'Wolfgang.Etl.Abstractions.Tests.Unit' not 'Wolfgang.Etl.Abstractions.Tests'.

Copilot · 2025-12-27T20:09:27Z

@Chris-Wolfgang I've opened a new pull request, #47, to work on those changes. Once the pull request is ready, I'll request review from you.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Updated Wolfgang.Etl.Abstractions.csproj to include net10.0 in the TargetFrameworks list, enabling support for .NET 10.0 alongside existing frameworks.

…actions into develop

Add net10.0 target framework while preserving out-of-support frameworks

…-time Fix incorrect test project path in release workflow

Fix PowerShell line continuation syntax in release workflow

Fix incorrect test project path in release workflow

Add .NET 10.0 target framework support

- Introduce security-scanning.yml with secret and dependency scanning jobs - Update release.yaml to publish NuGet symbol packages (*.snupkg) - Change docfx.json output key from "dest" to "output" - Add "Articles" section to toc.yml for improved navigation Add security scanning workflow and doc/CI improvements - Introduce security-scanning.yml with secret and dependency scanning jobs, scheduled daily and on push to main - Update release.yaml to publish NuGet symbol packages (*.snupkg) - Change docfx.json output key from "dest" to "output" - Add "Articles" section to toc.yml for better navigation

Copilot AI review requested due to automatic review settings December 27, 2025 17:28

Copilot started reviewing on behalf of Chris-Wolfgang December 27, 2025 17:28 View session

Copilot AI reviewed Dec 27, 2025

View reviewed changes

Chris-Wolfgang and others added 2 commits December 27, 2025 12:44

Update docfx_project/toc.yml

98a0d31

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Initial plan

9dcf9e5

Copilot AI mentioned this pull request Dec 27, 2025

Add .NET 10.0 target framework support #42

Merged

14 tasks

Initial plan

49f18aa

Copilot AI mentioned this pull request Dec 27, 2025

Add .NET 10.0 target framework support #43

Merged

15 tasks

Chris-Wolfgang and others added 3 commits December 27, 2025 12:59

Update docfx_project/docfx.json

03954e1

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Add net10.0 to TargetFrameworks in Wolfgang.Etl.Abstractions.csproj

70dc0c4

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Initial plan

389dfaf

Copilot AI mentioned this pull request Dec 27, 2025

Fix incorrect test project path in release workflow #44

Merged

7 tasks

Initial plan

bfca444

Copilot AI mentioned this pull request Dec 27, 2025

Fix PowerShell line continuation syntax in release workflow #45

Merged

14 tasks

Copilot AI and others added 3 commits December 27, 2025 18:02

Add net10.0 to TargetFrameworks in Wolfgang.Etl.Abstractions.csproj

6ad16b3

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Update .github/workflows/release.yaml

a328f52

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Initial plan

9256549

Copilot AI mentioned this pull request Dec 27, 2025

Fix incorrect test project path in release workflow #46

Merged

15 tasks

Copilot AI and others added 3 commits December 27, 2025 18:03

Fix test project path in release.yaml

1ee65ae

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Update docfx_project/docfx.json

4d1f47c

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Fix PowerShell line continuation in release.yaml

c277b7d

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Chris-Wolfgang requested a review from Copilot December 27, 2025 18:04

Copilot started reviewing on behalf of Chris-Wolfgang December 27, 2025 18:04 View session

Fix test project path in release.yaml workflow

3024ca2

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Copilot AI reviewed Dec 27, 2025

View reviewed changes

Initial plan

076ad7d

Copilot AI mentioned this pull request Dec 27, 2025

Add net10.0 target framework while preserving out-of-support frameworks #47

Merged

15 tasks

Chris-Wolfgang and others added 11 commits December 27, 2025 15:12

Update .github/workflows/release.yaml

fb0ec43

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Add net10.0 target framework to Wolfgang.Etl.Abstractions

eeeaca5

Co-authored-by: Chris-Wolfgang <210299580+Chris-Wolfgang@users.noreply.github.com>

Add net10.0 as a target framework

a780369

Updated Wolfgang.Etl.Abstractions.csproj to include net10.0 in the TargetFrameworks list, enabling support for .NET 10.0 alongside existing frameworks.

Merge branch 'develop' of https://github.com/Chris-Wolfgang/ETL-Abstr…

5d732f2

…actions into develop

Merge pull request #47 from Chris-Wolfgang/copilot/sub-pr-41-please-work

9f6137b

Add net10.0 target framework while preserving out-of-support frameworks

Merge pull request #46 from Chris-Wolfgang/copilot/sub-pr-41-one-more…

3f0f2d9

…-time Fix incorrect test project path in release workflow

Merge pull request #45 from Chris-Wolfgang/copilot/sub-pr-41-yet-again

5bb9bb2

Fix PowerShell line continuation syntax in release workflow

Merge pull request #44 from Chris-Wolfgang/copilot/sub-pr-41-another-one

1791b19

Fix incorrect test project path in release workflow

Merge pull request #43 from Chris-Wolfgang/copilot/sub-pr-41-again

04edcc3

Add .NET 10.0 target framework support

Merge pull request #42 from Chris-Wolfgang/copilot/sub-pr-41

5983f8a

Add .NET 10.0 target framework support

Chris-Wolfgang merged commit 47115e0 into main Dec 28, 2025
6 checks passed

dependabot Bot mentioned this pull request Mar 14, 2026

Bump the dotnet-dependencies group with 8 updates Chris-Wolfgang/ETL-Test-Kit#5

Closed

Conversation

Chris-Wolfgang commented Dec 27, 2025

Description

Type of change

How Has This Been Tested?

Checklist

Screenshots (if applicable)

Additional context

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Dec 27, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 27, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 27, 2025

Choose a reason for hiding this comment

Uh oh!

Chris-Wolfgang Dec 27, 2025

Choose a reason for hiding this comment

Uh oh!

Chris-Wolfgang Dec 27, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 27, 2025

Choose a reason for hiding this comment

Uh oh!

Chris-Wolfgang Dec 27, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Dec 27, 2025

Uh oh!

Copilot AI commented Dec 27, 2025

Uh oh!

Copilot AI commented Dec 27, 2025

Uh oh!

Copilot AI commented Dec 27, 2025

Uh oh!

Copilot AI commented Dec 27, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Dec 27, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Dec 27, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Dec 27, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants