Skip to content

CI Test#615

Merged
hueifeng merged 11 commits into
masterfrom
ci-test
Jun 7, 2026
Merged

CI Test#615
hueifeng merged 11 commits into
masterfrom
ci-test

Conversation

@hueifeng
Copy link
Copy Markdown
Member

@hueifeng hueifeng commented Jun 7, 2026

Testing CI workflow changes

@hueifeng hueifeng force-pushed the ci-test branch 24 times, most recently from 9244410 to 8e140f4 Compare June 7, 2026 09:09
@hueifeng
Copy link
Copy Markdown
Member Author

hueifeng commented Jun 7, 2026

@copilot review

Copy link
Copy Markdown
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 is focused on adjusting CI behavior and test execution stability, while also improving PDF native library loading (notably for macOS arm64) and adding test categorization for PDF-related tests.

Changes:

  • Overhauled the GitHub Actions workflow into a multi-job CI with OS/framework matrix testing, artifacts, and an “alls-green” gate.
  • Updated the test project to add GitHub Actions logging support, disable xUnit parallelism via xunit.runner.json, and adjust target frameworks (drop net7.0, add net9.0).
  • Added PDF test categorization traits and introduced a DllImportResolver bootstrap to improve wkhtmltox native resolution.

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
.github/workflows/dotnetcore.yml Reworked CI into change-detection + matrix test + final gate; adds log/artifact/diagnostic options.
src/Magicodes.ExporterAndImporter.Tests/xunit.runner.json Disables parallel test collections and limits threads to 1 for more deterministic runs.
src/Magicodes.ExporterAndImporter.Tests/Magicodes.IE.Tests.csproj Updates TFMs, adds GitHub Actions test logger package, and ensures xunit.runner.json is copied.
src/Magicodes.ExporterAndImporter.Tests/PdfExporter_Tests.cs Tags PDF exporter tests with a Category trait.
src/Magicodes.ExporterAndImporter.Tests/MagicodesMiddleware_Tests.cs Tags a PDF-related middleware test with a Category trait.
src/Magicodes.ExporterAndImporter.Tests/ExcelTemplateExporter_Tests.cs Simplifies pre-delete logic for generated output files in two tests.
src/Magicodes.ExporterAndImporter.Pdf/PdfNativeLibraryBootstrapper.cs Adds DllImportResolver registration to improve wkhtmltox native loading behavior and updates install guidance.
src/Magicodes.ExporterAndImporter.Pdf/PdfExporter.cs Ensures native bootstrap executes before converter usage.

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

Comment thread .github/workflows/dotnetcore.yml Outdated
Comment thread .github/workflows/dotnetcore.yml
Comment thread .github/workflows/dotnetcore.yml Outdated
Comment thread .github/workflows/dotnetcore.yml Outdated
Comment thread .github/workflows/dotnetcore.yml Outdated
hueifeng and others added 6 commits June 7, 2026 17:41
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Member Author

@hueifeng hueifeng left a comment

Choose a reason for hiding this comment

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

APPROVED

@hueifeng hueifeng merged commit 4c7073e into master Jun 7, 2026
59 of 61 checks passed
@hueifeng hueifeng deleted the ci-test branch June 7, 2026 10:42
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