Skip to content

Update textmate grammar to latest version - #9920

Merged
srawlins merged 3 commits into
flutter:masterfrom
DanTup:upadte-grammar
Jul 28, 2026
Merged

Update textmate grammar to latest version#9920
srawlins merged 3 commits into
flutter:masterfrom
DanTup:upadte-grammar

Conversation

@DanTup

@DanTup DanTup commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

This updates the grammar to the latest version from https://github.com/dart-lang/dart-syntax-highlight and regenerates the goldens.

(I have another upcoming change, but to simplify reviewing, I didn't want to mix it in with previous updates here)

This updates the grammar to the latest version from https://github.com/dart-lang/dart-syntax-highlight and regenerates the goldens.

(I have another upcoming change, but to simplify reviewing, I didn't want to mix it in with previous updates here)
@DanTup
DanTup requested a review from a team as a code owner July 28, 2026 13:48
@DanTup
DanTup requested review from bkonyi and removed request for a team July 28, 2026 13:48

@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 updates the Dart syntax highlighting configuration (dart_syntax.json) to version 1.5.0, introducing a new pattern for class identifiers with optional factory methods, updating the highlighting of the new keyword, and updating the corresponding golden test files. A review comment points out that the renaming of the new keyword scope from keyword.control.new.dart to keyword.new.dart requires a corresponding update in syntax_highlighter.dart to prevent highlighting issues.

Comment thread packages/devtools_app/assets/dart_syntax.json
@DanTup
DanTup requested a review from srawlins as a code owner July 28, 2026 13:52
@srawlins
srawlins merged commit 1681401 into flutter:master Jul 28, 2026
93 of 94 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