meilisearch-rust icon indicating copy to clipboard operation
meilisearch-rust copied to clipboard

Add cargo fmt to the CI

Open irevoire opened this issue 3 years ago • 1 comments

We should add cargo fmt to the CI once meilisearch/meilisearch-rust#289 is merged

irevoire avatar May 25 '22 17:05 irevoire

As #289 has been closed, a PR to format the code is required before adding cargo fmt to the CI.

For those who wants to contribute to the formating of this repo you can re-use the code from this PR #289. They have been mostly reviewed and only required small additional changes

bidoubiwa avatar Jun 06 '22 10:06 bidoubiwa

Hi! Can you explain this issue a bit more? Maybe I will be able to take it

ghost avatar Oct 06 '22 01:10 ghost

cargo fmt checks the style of your code. The issue must be updated as the PR that was supposed to format the code was closed.

So the idea is

  • [ ] format the code using cargo fmt
  • [ ] Add format test in the tests.yml ci

bidoubiwa avatar Oct 06 '22 08:10 bidoubiwa

See example here

bidoubiwa avatar Oct 06 '22 09:10 bidoubiwa

Thanks a bunch, @bidoubiwa!

ghost avatar Oct 06 '22 09:10 ghost