Skip to content

[api_summary] Alphabetize meta annotation getters, models, and JSON serialization - #2512

Closed
kevmoo wants to merge 2 commits into
api-summary-meta-annotationsfrom
api-summary-meta-annotations-sorting
Closed

kevmoo wants to merge 2 commits into
api-summary-meta-annotationsfrom
api-summary-meta-annotations-sorting

Conversation

@kevmoo

@kevmoo kevmoo commented Aug 13, 2026

Copy link
Copy Markdown
Member

Alphabetize contractual package:meta annotation getters, models, toJson serialization keys, and api.txt parentheticals for consistency and canonical output.

Stacked on #2511.

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request performs a comprehensive alphabetization refactoring across multiple files in the api_summary package, reordering fields, properties, and methods alphabetically to improve consistency and readability. A review of the changes reveals that the documentation comments for isConst and isEnumConstant in ApiExecutable were accidentally removed during the reordering in pkgs/api_summary/lib/src/api_declaration.dart. It is recommended to restore these comments to comply with the repository style guide requiring public members to be documented.

Comment thread pkgs/api_summary/lib/src/api_declaration.dart
@kevmoo
kevmoo requested a review from natebosch August 13, 2026 19:50
@kevmoo
kevmoo force-pushed the api-summary-meta-annotations-sorting branch from dc87a2c to 3bce624 Compare August 13, 2026 23:41
@kevmoo
kevmoo deleted the branch api-summary-meta-annotations August 13, 2026 23:53
@kevmoo kevmoo closed this Aug 13, 2026
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