Skip to content

Order configuration properties alphabetically - #5151

Merged
arturcic merged 1 commit into
mainfrom
refactor/order-configuration-properties
Aug 18, 2026
Merged

Order configuration properties alphabetically#5151
arturcic merged 1 commit into
mainfrom
refactor/order-configuration-properties

Conversation

@arturcic

Copy link
Copy Markdown
Member

Summary

  • serialize configuration properties alphabetically at every configuration level
  • preserve intentional dictionary ordering for branch names and custom merge-message formats
  • order generated configuration schema properties by their serialized names
  • align reference documentation, workflow examples, and approved snapshots

Stack

  1. This preparation PR
  2. #5147 - ignore tags
  3. #5148 - ignore branches

Validation

  • dotnet build src/GitVersion.slnx --no-restore
  • dotnet format --verify-no-changes src/GitVersion.slnx --no-restore
  • dotnet test src/GitVersion.slnx --no-build - 37,202 passed
  • regenerated only schemas/7.0/GitVersion.configuration.json

Copilot AI lite review requested due to automatic review settings August 18, 2026 10:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sonarqubecloud

Copy link
Copy Markdown

@mergify mergify Bot mentioned this pull request Aug 18, 2026
4 tasks
@arturcic arturcic mentioned this pull request Aug 18, 2026
4 tasks
@arturcic
arturcic merged commit 464868d into main Aug 18, 2026
66 checks passed
@arturcic
arturcic deleted the refactor/order-configuration-properties branch August 18, 2026 11:46
@mergify

mergify Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Thank you @arturcic for your contribution!

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.

3 participants