Page MenuHomePhabricator

[Chore] Lint services/tunnelbroker/CMakeLists.txt
ClosedPublic

Authored by jon on Jul 18 2022, 4:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jun 12, 9:35 PM
Unknown Object (File)
Thu, Jun 6, 6:29 PM
Unknown Object (File)
Thu, Jun 6, 1:00 PM
Unknown Object (File)
Thu, Jun 6, 1:27 AM
Unknown Object (File)
Wed, Jun 5, 7:21 PM
Unknown Object (File)
Wed, Jun 5, 6:15 PM
Unknown Object (File)
Wed, Jun 5, 3:17 PM
Unknown Object (File)
Tue, Jun 4, 5:50 PM

Details

Summary

Lint tunnelbroker's cmake

Test Plan
nix develop
cmake-lint services/backup/CMakeLists.txt

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 18 2022, 4:27 PM
Harbormaster failed remote builds in B10603: Diff 14596!

Apply refactor throughout referrers

This revision is now accepted and ready to land.Jul 18 2022, 6:00 PM

Small note on this stack – personally I probably would've put all of the lint results into a single diff. We generally prefer small diffs, but it's really all about making life easier for the reviewer. When there are code changes, separating into smaller diffs makes review easier, but sometimes large diffs can be easier to review.

noted, guess for this one there likely wouldn't be much commentary anyway.