Page MenuHomePhabricator

[services] fix rust-pre-commit hook
ClosedPublic

Authored by varun on Jun 30 2022, 2:16 PM.
Tags
None
Referenced Files
F3502651: D4423.diff
Fri, Dec 20, 4:34 AM
F3500481: D4423.id14085.diff
Fri, Dec 20, 1:42 AM
F3500480: D4423.id14056.diff
Fri, Dec 20, 1:42 AM
F3500455: D4423.id.diff
Fri, Dec 20, 1:42 AM
F3500436: D4423.diff
Fri, Dec 20, 1:38 AM
Unknown Object (File)
Thu, Dec 19, 1:55 AM
Unknown Object (File)
Thu, Dec 19, 1:55 AM
Unknown Object (File)
Tue, Dec 17, 7:00 AM

Details

Summary

we add the "all" flag to format all packages and their local dependencies. we add the "check" option to run rustfmt in check mode, which ensures that the pre-commit hook fails further formatting is required.

Test Plan

tried unformatting some code in the commtest package and then committing my changes. the linter stopped me from committing the code, as expected

Diff Detail

Repository
rCOMM Comm
Branch
fix-cargo-fmt-hook
Lint
No Lint Coverage
Unit
No Test Coverage