meilisearch/meilisearch-rust

Add cargo fmt to the CI

irevoire opened this issue · 5 comments

irevoire commented

We should add cargo fmt to the CI once #289 is merged

bidoubiwa commented

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

Deleted user commented

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

bidoubiwa commented

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
Deleted user commented

Thanks a bunch, @bidoubiwa!