Fix cargo clippy && fmt #302

zhiburt · 2021-10-01T23:09:39Z

For some reason cargo fmt changes showed like they changed entire file where's in reality is 2 lines....
It showed better in vscode diff.

Kampfkarren · 2021-10-02T02:19:31Z

For some reason cargo fmt changes showed like they changed entire file where's in reality is 2 lines....

~~It's the newlines, can you try to preserve them if possible?~~ Actually no, you're good, it's converting them to LF.

Kampfkarren · 2021-10-02T02:19:53Z

selene-lib/src/ast_util/visit_nodes.rs

@@ -46,6 +46,7 @@ macro_rules! make_node_visitor {
                )+
            }

+            #[allow(clippy::enum_variant_names)]


What's this for?

Oh, because they're camel case...I wonder why I did that.

Oh, I see! Fair enough.

fix cargo clippy && fmt

86257e7

Kampfkarren reviewed Oct 2, 2021

View reviewed changes

Kampfkarren merged commit 917e7aa into Kampfkarren:main Oct 2, 2021

Provide feedback