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.

Sign up for GitHub

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

Jump to bottom

rustfmt no longer builds after rust-lang/rust#71059 #71077

Closed
rust-highfive opened this issue Apr 12, 2020 · 1 comment · Fixed by racer-rust/racer#1104 or #71159
Closed

rustfmt no longer builds after rust-lang/rust#71059 #71077

rust-highfive opened this issue Apr 12, 2020 · 1 comment · Fixed by racer-rust/racer#1104 or #71159
Assignees
@topecongiro

Comments

@rust-highfive
Copy link
Collaborator

Hello, this is your friendly neighborhood mergebot.
After merging PR #71059, I observed that the tool rustfmt no longer builds.
A follow-up PR to the repository https://github.com/rust-lang/rustfmt is needed to fix the fallout.

cc @Dylan-DPC, do you think you would have time to do the follow-up work?
If so, that would be great!

And nominating for compiler team prioritization.
@phansch
Copy link
Member

phansch commented Apr 13, 2020

Probably caused by #71029

phansch added a commit to phansch/rust-clippy that referenced this issue Apr 13, 2020
@phansch
Temporarily disable rustfmt integration test
8ad1d78 
Running rustfmt from master is currently broken:
rust-lang/rust#71077
@phansch phansch mentioned this issue Apr 13, 2020
Merged
bors added a commit to rust-lang/rust-clippy that referenced this issue Apr 13, 2020
@bors
Auto merge of #5461 - phansch:disable_rustfmt_integration_test, r=mat…
6d4cc56 
…thiaskrgr

Temporarily disable rustfmt integration test

Running rustfmt from master is currently broken and [fails our bors build](https://github.com/rust-lang/rust-clippy/runs/582066368#step:10:19):
rust-lang/rust#71077

changelog: none
@topecongiro topecongiro mentioned this issue Apr 14, 2020
Merged
@AndrewGaspar AndrewGaspar mentioned this issue Apr 14, 2020
Merged
@topecongiro topecongiro mentioned this issue Apr 15, 2020
Merged
Centril added a commit to Centril/rust that referenced this issue Apr 15, 2020
@Centril
Rollup merge of rust-lang#71159 - topecongiro:rustfmt-1.4.14, r=Dylan…
128d622 
…-DPC

Bump rustfmt and rls

Close rust-lang#71076. Close rust-lang#71077.

r? @Xanewok
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this issue Apr 15, 2020
@Dylan-DPC
Rollup merge of rust-lang#71159 - topecongiro:rustfmt-1.4.14, r=Dylan…
2be7b4d 
…-DPC

Bump rustfmt and rls

Close rust-lang#71076. Close rust-lang#71077.

r? @Xanewok
@bors bors closed this as completed in be895b2 Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@topecongiro topecongiro

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bump rustc-ap-* crates to 654.0.0 topecongiro/racer
Bump rustfmt and rls topecongiro/rust
3 participants
@phansch @rust-highfive @topecongiro