Skip to content

Conversation

@mattleibow
Copy link
Member

Description of Change

Things are coming. Sorting is making the diffs easier to grep as well as to spot related changes.

Copilot AI review requested due to automatic review settings July 16, 2025 16:09
@mattleibow mattleibow requested a review from a team as a code owner July 16, 2025 16:09
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 pull request sorts the public API files across all platform targets to improve diff readability and make it easier to identify related changes. The changes reorganize the API entries alphabetically while preserving the exact same API surface - no functional changes are made.

  • Sorts API entries alphabetically in PublicAPI.Unshipped.txt files
  • Affects multiple platform targets (netstandard, net, net-windows, net-tizen, net-maccatalyst, net-ios, net-android)
  • Improves maintainability by making diffs more readable and grouping related APIs together

Reviewed Changes

Copilot reviewed 30 out of 30 changed files in this pull request and generated no comments.

File Description
src/Essentials/src/PublicAPI/*/PublicAPI.Unshipped.txt Sorts HapticFeedback and Vibration IsSupported properties alphabetically
src/Core/src/PublicAPI/*/PublicAPI.Unshipped.txt Reorganizes extensive API lists alphabetically, moving type converters, handlers, and other APIs to their sorted positions
src/Core/maps/src/PublicAPI/*/PublicAPI.Unshipped.txt Minor reordering of MapElementHandlerUpdate.Equals method

@mattleibow mattleibow merged commit 4566c87 into net10.0 Jul 16, 2025
111 of 129 checks passed
@mattleibow mattleibow deleted the dev/sort-apis branch July 16, 2025 18:48
@github-actions github-actions bot locked and limited conversation to collaborators Aug 16, 2025
@PureWeen PureWeen added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Aug 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants