search

rust-analyzer / rowan

Apache License 2.0
697 stars 57 forks source link

Add cargo fmt to travis build config #65

Closed Atul9 closed 4 years ago

Atul9 commented 4 years ago

Note: To fix the cargo fmt build error the https://github.com/rust-analyzer/rowan/pull/64 PR needs to be merged first.

matklad commented 4 years ago

Let's add this as a test instead, I am not a fan of encoding stuff into the CI config. See rust-analyzer as an example:

https://github.com/rust-analyzer/rust-analyzer/blob/7488cd6a1b3604d6b4f11a7f83b6006a34cda0c0/xtask/tests/tidy.rs#L33-L38

Atul9 commented 4 years ago

@matklad ok, I am checking ^ rust-analyzer run_rustfmt & other functions which are called.
I am new to this project & rust-analyzer. In which file should it be added in this project?