Disable rustfmt running on AppVeyor for now

[phansch]

rustfmt doesn't seem to execute correctly right now, causing some
backlog of PRs. It's not critical that it runs on Windows, so I've
disabled it for now.

changelog: none

[Manishearth]

@bors r+

[bors]

📌 Commit 0703ec8 has been approved by Manishearth

[bors]

⌛ Testing commit 0703ec8 with merge 9f758b7...

[bors]

💔 Test failed - checks-travis

[ghost]

The reason CI is failing is because rustup-toolchain-install-master now includes cargo by default. We're altering the path to link directly to the toolchain here so it's using that instead of the rustup proxy.

[llogiq]

@bors r=@Manishearth retry

[bors]

💡 This pull request was already approved, no need to approve it again.

• This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.

[bors]

📌 Commit 0703ec8 has been approved by Manishearth

[bors]

⌛ Testing commit 0703ec8 with merge e514587...

[llogiq]

@bors retry

[mati865]

@mikerite Cargo path LGTM:

which cargo
/c/Users/appveyor/.cargo/bin/cargo

https://ci.appveyor.com/project/mati865/rust-clippy-k06j7

[bors]

⌛ Testing commit 0703ec8 with merge 5845b90...

[llogiq]

This was missing the 'changelog' line.

[tesuji]

Do we ever need to run rustfmt on Windows? IIRC rustfmt works consistent on all platforms.
So running rustfmt on Linux CI is enough.

[llogiq]

I think even if there were differences, we won't likely care about them. This is clippy's CI, not rustfmt's.

[bors]

💔 Test failed - status-appveyor

[ghost]

If a Windows user ever wanted to contribute it would be nice if they could run the rustfmt test. There is little chance of that if we can't even get it working on CI.

[llogiq]

The problem appears to be with rustup, not rustfmt.

[mati865]

@matthiaskrgr probably found the reason, check discord #4620

[ghost]

Too tired. Ignore my last few posts.

[mati865]

FYI it has been reported upstream rust-lang/cargo#7475