Page MenuHomePhabricator

[services] fix rust-pre-commit hook
ClosedPublic

Authored by varun on Jun 30 2022, 2:16 PM.
Tags
None
Referenced Files
F3504649: D4423.id14056.diff
Fri, Dec 20, 9:51 AM
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

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
Lint
Lint Not Applicable
Unit
Tests Not Applicable