Page MenuHomePhabricator

[services] fix rust-pre-commit hook
ClosedPublic

Authored by varun on Jun 30 2022, 2:16 PM.
Tags
None
Referenced Files
F2204530: D4423.id14085.diff
Sat, Jul 6, 4:50 PM
F2204529: D4423.id14056.diff
Sat, Jul 6, 4:50 PM
F2204528: D4423.id.diff
Sat, Jul 6, 4:49 PM
F2204527: D4423.diff
Sat, Jul 6, 4:49 PM
Unknown Object (File)
Wed, Jul 3, 5:07 AM
Unknown Object (File)
Sun, Jun 30, 9:23 AM
Unknown Object (File)
Sun, Jun 30, 9:23 AM
Unknown Object (File)
Sun, Jun 30, 9:22 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