Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release syntax aware heuristic expansion of diagnostic excerpts #22858

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

mgsloan
Copy link
Contributor

@mgsloan mgsloan commented Jan 8, 2025

Implementation PR was #21942

Release Notes:

  • Improved diagnostic excerpts by using syntactic info to determine the context lines to show.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jan 8, 2025
@mgsloan mgsloan enabled auto-merge January 8, 2025 20:12
@mgsloan mgsloan force-pushed the release-ast-aware-diagnostic-excerpts branch 2 times, most recently from 1934812 to 87975a3 Compare January 8, 2025 20:28
Release Notes:

- Improved diagnostic excerpts by using syntactic info to determine the context
lines to show.
@mgsloan mgsloan force-pushed the release-ast-aware-diagnostic-excerpts branch from 87975a3 to 0463302 Compare January 8, 2025 20:39
@mgsloan mgsloan added this pull request to the merge queue Jan 8, 2025
Merged via the queue into main with commit fe35695 Jan 8, 2025
13 checks passed
@mgsloan mgsloan deleted the release-ast-aware-diagnostic-excerpts branch January 8, 2025 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant