Add .rustfmt.toml to repo + format code about pelite HOT 7 CLOSED

I can confirm that this is not the case, all files changed by cargo +nightly fmt (idk if nightly matters here) are changed from crlf to lf. Look like it is an open bug in rustfmt: rust-lang/rustfmt#4097

Some git magic later it's all fixed but these are the kind of rough edges that confirm my biases against rustfmt 😛

from pelite.

It took some time but it's there

from pelite.

The other option is to disable rustfmt project-wide with disable_all_formatting in the .rustfmt.toml, though I'd suggest that it's better to pick a suboptimal style rather than ad-hoc style chosen by contributors

from pelite.

I would PR this, but it would be difficult to review due to the large number of changes, so I'll leave it up to the maintainer to push a version with cargo fmt run on it

from pelite.

Thanks for your consideration! I've been working on implementing a rustfmt policy that I can stomach.

I'm not a fan of autoformatters in general as they tend to make everything look consistently ugly.
But with some okay settings, a nightly rustfmt, refactoring some code and liberal application of rustfmt::skip I think I can make it work.

I'll try to get a PR up in a few more hours.

from pelite.

Good lord I just noticed rustfmt rewrote everything to lf which conflicts with my windows checkout and usage of autocrlf setting... I'll try to keep going and get the repo to cleanly pass rustfmt but rustfmt is just not ready for actual usage...

from pelite.

You can adjust this with the newline_style setting.

Looks like the default is "auto" though which tries to use whatever that file already uses, so your file probably already had a LF present.

from pelite.