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

rustfmt 1.x bump rustc-ap to v642 #4043

Merged
merged 3 commits into from
Feb 10, 2020

Conversation

calebcartwright
Copy link
Member

@calebcartwright calebcartwright commented Feb 9, 2020

Bumps the rustc-ap deps to v642 for the latest 1.x version of rustfmt

Refs rust-lang/rust#68917
Resolves #3903
Resolves #4009

Backport the fix for the parser bug where the messages from
fatal/non-recoverable parser errors were being silently eaten by
rustfmt.
@topecongiro topecongiro changed the base branch from rustfmt-1.4.11 to rustfmt-1.4.12 February 10, 2020 04:33
@topecongiro
Copy link
Contributor

@calebcartwright Thank you for your work! I was about to ask whether you could make a pull request that backports the #4022 to the rustfmt-1.4.11 branch, but then I found out that you have already created the PR! Thanks a million.

I changed the base branch to rustfmt-1.4.12, which is based on rustfmt-1.4.11, to keep the branch in the correct state.

@calebcartwright
Copy link
Member Author

@topecongiro when you get a chance can you publish v1.4.12 to crates.io?

I've gotten Racer bumped to rustc-ap-* v642 as well (racer-rust/racer#1091), and IIUC once v1.4.12 is published I can open a PR to update Rls so that Rls and rustfmt can then be fixed and available again on nightlies.

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