Skip to content

Update tree-sitter-dart#910

Merged
jasonforal merged 2 commits into
mainfrom
jf/dart
May 12, 2026
Merged

Update tree-sitter-dart#910
jasonforal merged 2 commits into
mainfrom
jf/dart

Conversation

@jasonforal

Copy link
Copy Markdown
Collaborator

Notes

Copilot AI review requested due to automatic review settings May 12, 2026 20:17
@jasonforal
jasonforal requested a review from a team as a code owner May 12, 2026 20:17

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.

Pull request overview

Updates the vendored Tree-sitter Dart grammar source used by static-analysis-kernel’s build script, switching from a forked repository/commit to the official upstream repository now that ABI 14-only constraints are no longer needed (per the notes referencing #909).

Changes:

  • Switch tree-sitter-dart repository URL from the fork to UserNobody14/tree-sitter-dart.
  • Update the pinned Dart grammar commit hash to 02e9bb191c7cb6c73cbd9e09a5d089e4496f9cff.
  • Remove fork/ABI-14-specific inline comments that are no longer applicable.

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

@datadog-official

Copy link
Copy Markdown

🎯 Code Coverage (details)
Patch Coverage: -1.00%
Overall Coverage: 85.17% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 516bc75 | Docs | Datadog PR Page | Give us feedback!

@jasonforal
jasonforal merged commit aa0369b into main May 12, 2026
94 checks passed
@jasonforal
jasonforal deleted the jf/dart branch May 12, 2026 20:59
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