Skip to content

Conversation

@HeDo88TH
Copy link
Collaborator

  • Added syntactic sugar
  • Minor code refactor and cleanup
  • Upgrade to .NET 9
  • Upgraded nuget packages
  • Got rid of FluentAssertions. Welcome Shoudly!
  • Version bump to v1.1.0

@HeDo88TH HeDo88TH self-assigned this Aug 22, 2025
Copilot AI review requested due to automatic review settings August 22, 2025 23:56
@HeDo88TH HeDo88TH added the enhancement New feature or request label Aug 22, 2025
Copy link

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 performs a comprehensive upgrade and refactoring of the Obj2Tiles project, upgrading from .NET 8 to .NET 9 while implementing modern C# syntax patterns. The changes include dependency updates, code modernization using collection expressions and simplified syntax, and a switch from FluentAssertions to Shouldly for testing.

  • Upgraded .NET framework from 8.0 to 9.0 across all projects and CI workflows
  • Modernized C# syntax using collection expressions ([]) and simplified patterns
  • Replaced FluentAssertions with Shouldly testing library and updated NuGet packages

Reviewed Changes

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

Show a summary per file
File Description
Multiple .csproj files Updated target framework from net8.0 to net9.0 and version bumps
Multiple .cs files Applied modern C# syntax including collection expressions and simplified patterns
Test files Migrated from FluentAssertions to Shouldly assertions
Publish profiles Updated framework references for deployment targets
GitHub workflows Updated CI/CD to use .NET 9 and newer action versions

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

@HeDo88TH HeDo88TH merged commit 56b236b into master Aug 23, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants