Run Rustfmt on entire repo

[josephlr]

This PR keeps allow_failures in the Travis presubmit, we should just run rustfmt from time to time.

[newpavlov]

Personally I am not sure about removing allow_failures, in my experience it results in a more annoyance than it's worth.

[josephlr]

Personally I am not sure about removing allow_failures, in my experience it results in a more annoyance than it's worth.

I mean all it does is force PRs to be properly formatted when they are submitted. Or is it more a concern about rustfmt's stability?

[newpavlov]

Both, I had experienced CI breakage on my projects due to rustfmt changes, and it will be annoying for contributors who don't use rustfmt. So personally I think it will be better to allow rustfmt failure and fix formatting manually from time to time.

[josephlr]

@newpavlov that makes sense, fixed