Bring rustfmt back to pre-commit about nearcore HOT 9 CLOSED

cc: @nearmax
it would be nice to have the option for all members to be on one line and only wrap starting at method's.

-                message.body
-                       .parents
+                message.body.parents
                        .into_iter()
                        .filter(|h| !self.dag.expect(UNINITIALIZED_DAG_ERR).contains_message(&h))
                        .collect();

relevant discussion and links:
rust-lang/rustfmt#3210
rust-lang/style-team#66
rust-lang/style-team@bec533c

this is not possible with rustfmt and it seems like they are opposed to adding the option

from nearcore.

The linter gets quite terrible:
https://gitlab.com/nearprotocol/nearcore/-/jobs/123228836
If we want linter and it does not allow us to turn this wrapping off then we should disable the linter and fork it. This should not be blocking us

-        let (v0,v1,v2,v3,v4,v5,v6,v7,v8,v9,v10,v11,v12,v13,v14,v15,v16,v17,v18,v19,v20,v21,v22,v23);
+        let (
+            v0,
+            v1,
+            v2,
+            v3,
+            v4,
+            v5,
+            v6,
+            v7,
+            v8,
+            v9,
+            v10,
+            v11,
+            v12,
+            v13,
+            v14,
+            v15,
+            v16,
+            v17,
+            v18,
+            v19,
+            v20,
+            v21,
+            v22,
+            v23,
+        );

from nearcore.

@azban we are ok with clippy / cargo fmt for now?

from nearcore.

Seems clipply disabled on nightly. I'd prefer to get it back

from nearcore.

Last time I checked clippy did not compile on CI because of the nightly switch.

from nearcore.

@bowenwang1996 fixed that in #675

i'm fine with clippy.. we still haven't converged on cargo fmt for linting, and i feel like it would be a good idea to have a strict formatting tool to use at some point

from nearcore.

I disabled clippy again due to its instability with nightly. On some days it work with nightly and when we are not lucky, it doesn't.

from nearcore.

@azban I am fine with rustfmt. Sometimes it derps, but it seems to be the widely accepted standard and we should be using at least something, so let's use it.

from nearcore.

Closing this, as we have now CI check.

from nearcore.