search

clojure-rs / ClojureRS

Clojure, implemented atop Rust (unofficial)
Apache License 2.0
952 stars 27 forks source link

Automate the use of rustfmt in github actions #77

Open zackteo opened 4 years ago

zackteo commented 4 years ago

Ensure formatting is kept consistent, automating that process. Not sure if there's a point to adding clippy into the mix too

naomijub commented 4 years ago

Maybe this helps? https://github.com/clojure-rs/ClojureRS/pull/78

Github actions will fail if it code is not correctly formatted. You could add cargo-husky to prevent committing unformatted code example

zackteo commented 4 years ago

Cool! Actually, do you know if rustfmt or clippy is the standard? :o

naomijub commented 4 years ago

It is supposed to be rustfmt, but I don't know if they have merged as of now

naomijub commented 2 years ago

Rust is a formatter while clippy is a linter:

I can apply clippy in a next PR as the project os too large to fix it all at once.