Skip to content

Use new spanned diagnostics #320

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

Merged
merged 4 commits into from
Jul 21, 2025
Merged

Conversation

oli-obk
Copy link
Contributor

@oli-obk oli-obk commented Jun 23, 2025

This will also use the latest spanned crate, so even with debug prints one won't get the [0..0] byte offset messages anymore

Diagnostics now look like

image

pointing to both the source code that emitted the error as well as the processed text during whose processing the error occurred

@oli-obk oli-obk force-pushed the annotate-snippets branch 2 times, most recently from 1b66c69 to f9bf422 Compare June 25, 2025 13:52
@oli-obk oli-obk force-pushed the annotate-snippets branch from f9bf422 to 909a2b1 Compare June 30, 2025 13:12
Copy link
Contributor

@nikomatsakis nikomatsakis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, spanned looks cool! Merging.

@oli-obk
Copy link
Contributor Author

oli-obk commented Jul 21, 2025

looks like I need to rebase

@nikomatsakis
Copy link
Contributor

nikomatsakis commented Jul 21, 2025

@oli-obk I think I fixed it?

@oli-obk
Copy link
Contributor Author

oli-obk commented Jul 21, 2025

oh that works, too XD but I do have a rebased version if you prefer

@nikomatsakis
Copy link
Contributor

@oli-obk sure, go ahead and push, then I'll merge

@nikomatsakis nikomatsakis merged commit 52d766a into rust-lang:main Jul 21, 2025
2 checks passed
@nikomatsakis
Copy link
Contributor

@oli-obk I got impatient

@oli-obk oli-obk deleted the annotate-snippets branch July 21, 2025 17:12
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