cargo-crev icon indicating copy to clipboard operation
cargo-crev copied to clipboard

chore: reformat using `cargo +nightly fmt --all`

Open eirnym opened this issue 8 months ago • 3 comments

Checklist:

  • [x] cargo +nightly fmt --all
  • [ ] Modify CHANGELOG.md if applicable

eirnym avatar Apr 27 '25 16:04 eirnym

Questions to maintainers:

  • Should I create CI workflow to enforce pre-commit run?
  • Should I reformat toml files using taplo as well?
  • if taplo is advised, should I add it to pre-commit hook?

eirnym avatar Apr 27 '25 16:04 eirnym

I already started using taplo with autoformatting in my projects and I think it's a nice improvement tying into how i like uniform Rust code formatting, so :+1: from me, but ping @kornelski .

dpc avatar Jun 18 '25 16:06 dpc

In my projects I use pre-commit hook to format code and toml, as well as check it on CI using the same pre-commit. You can check how I've done it here. In my CI i wait for all checks to be done, which is a separate small CI job.

eirnym avatar Jun 18 '25 17:06 eirnym