You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The last few weeks I have been trying to get suggestions working within annotate-snippets, and have been able to match rustc suggestion output in most cases. Most of my time was spent trying to understand the different rendering styles for suggestions, and how they should be adapted to fit within annotate-snippets.
I have been a bit busy over the past two months, so I haven't had as much time to work on annotate-snippets as I had hoped. That said, progress was still being made!
I continued working on suggestions, but as I got deeper into the implementation, I realized getting the corner cases correct would be very difficult with the annotate-snippets architecture. I started work on a more direct translation from rustc, and quickly ended up with something that closely matched it at the cost of significant differences from how annotations are rendered. These differences, and some problems I noticed with the API for suggestions, led to a discussion with @epage about how to resolve these problems. During the discussion, I realized it would be beneficial if the internals of annotate-snippets resembled rustc as it would make it easier for contributors to rustc to contribute to annotate-snippets. @epage was helpful enough to design a new API, which resolved my suggestion API concerns and should make it easier for annotate-snippets's internals to resemble rustc. Initial work on the new design has yielded positive results, so I am hopeful this was a good decision. During all of this, I started to work on a tool that should help convert some of rustc's UI tests to be used in annotate-snippet's testsuite.
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Summary
Switch to annotate-snippets for rendering rustc's output, with no loss of functionality or visual regressions.
Tasks and status
The text was updated successfully, but these errors were encountered: