Skip to content

Conversation

@stesee
Copy link
Contributor

@stesee stesee commented Aug 17, 2025

No description provided.

Copilot AI review requested due to automatic review settings August 17, 2025 06:40
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 is a housekeeping PR focused on updating dependencies, modernizing code practices, and streamlining the build process. The changes include package updates, dependency migrations, and workflow improvements.

  • Updated multiple NuGet packages to newer versions across test and main projects
  • Migrated from SixLabors.ImageSharp to SkiaSharp for image processing
  • Improved code quality with better documentation, error handling, and class design

Reviewed Changes

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

Show a summary per file
File Description
OpenXmlToHtmlTests/OpenXmlToHtmlTests.csproj Updated test SDK and analyzer packages
OpenXmlToHtmlTests/OpenXmlToHtmlIntegrationTests.cs Migrated from ImageSharp to SkiaSharp for bitmap validation
OpenXmlToHtmlOpenApiTests/OpenXmlToHtmlOpenApiTests.csproj Updated AngleSharp, test SDK, and test runner packages
OpenXmlToHtmlCli/Program.cs Made Program class static since it only contains static methods
OpenXmlToHtmlCli/OpenXmlToHtmlCli.csproj Updated SonarAnalyzer and ILLink packages
OpenXmlToHtml/Tooling/Linux.cs Added XML documentation and made class static with better exception handling
OpenXmlToHtml/OpenXmlToHtml.csproj Updated OpenXmlPowerTools and SonarAnalyzer packages
OpenXmlToHtml/ExportImageHandler.cs Added input validation and improved null checking
.vscode/settings.json Added test discovery configuration
.github/workflows/dotnet.yml Replaced third-party zip action with native PowerShell command and removed Azure deployment job

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@stesee stesee merged commit a4f87a5 into main Aug 17, 2025
9 checks passed
@stesee stesee deleted the HouseKeeping branch August 17, 2025 07:24
@github-actions github-actions bot locked and limited conversation to collaborators Aug 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants