Skip to content

Comments

Upgrade v1.8.0#78

Merged
prozolic merged 1 commit intomainfrom
v1.8.0
Dec 6, 2025
Merged

Upgrade v1.8.0#78
prozolic merged 1 commit intomainfrom
v1.8.0

Conversation

@prozolic
Copy link
Owner

@prozolic prozolic commented Dec 6, 2025

Copilot AI review requested due to automatic review settings December 6, 2025 09:20
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 PR upgrades CsToml from v1.7.5 to v1.8.0, introducing new features and improving documentation. The upgrade includes support for array serialization styles, updates to the attribute API syntax, and various documentation improvements.

Key changes:

  • Version number bumped from 1.7.5 to 1.8.0 in NuGet.props
  • New ArrayStyle serialization option for controlling collection serialization format
  • API change: aliasName constructor parameter replaced with AliasName property in [TomlValueOnSerialized]
  • Documentation improvements with grammar corrections and clarifications
  • Updated benchmark image and references throughout documentation

Reviewed changes

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

Show a summary per file
File Description
src/NuGet.props Version number updated from 1.7.5 to 1.8.0
img/benchmark_serialization.png Updated benchmark visualization image (binary file)
README.md Comprehensive documentation updates including version references, API syntax changes, new ArrayStyle feature documentation, grammar improvements, and expanded code examples
CLAUDE.md Updated language version reference and minor formatting cleanup
.github/workflows/nuget-pack-push.yml Refactored pack command to explicitly specify each project file instead of using wildcard

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

Copy link

Copilot AI commented Dec 6, 2025

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

@prozolic prozolic merged commit 72ad590 into main Dec 6, 2025
7 checks passed
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