-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Refactor pretty printing to use the compiler API #33119
Conversation
r? @arielb1 (rust_highfive has picked a reviewer for you, use r? to override) |
This is a [breaking-change] if you are using the compiler API (I guess that should be in a commit message). |
☔ The latest upstream changes (presumably #33089) made this pull request unmergeable. Please resolve the merge conflicts. |
rebased r? @pnkfelix |
☔ The latest upstream changes (presumably #33296) made this pull request unmergeable. Please resolve the merge conflicts. |
@nrc r=me sorry I didn't get to it before it hit a new conflict |
This commit still does a lot of building in pretty because we always run after parsing.
@bors: r=pnkfelix |
📌 Commit c1c6e99 has been approved by |
Refactor pretty printing to use the compiler API
No description provided.