Page MenuHomePhabricator

[identity] some nits
ClosedPublic

Authored by varun on Apr 24 2023, 1:51 PM.
Tags
None
Referenced Files
F1439515: D7596.id25644.diff
Thu, Mar 28, 7:41 PM
F1439513: D7596.id25628.diff
Thu, Mar 28, 7:41 PM
F1439493: D7596.id.diff
Thu, Mar 28, 7:41 PM
F1439457: D7596.diff
Thu, Mar 28, 7:38 PM
F1432772: D7596.diff
Thu, Mar 28, 5:42 AM
Unknown Object (File)
Sun, Mar 17, 11:46 PM
Unknown Object (File)
Tue, Mar 12, 6:14 PM
Unknown Object (File)
Tue, Mar 12, 3:47 PM
Subscribers

Details

Summary

small changes to satisfy cargo clippy

Depends on D7595

Test Plan

cargo build

Diff Detail

Repository
rCOMM Comm
Branch
identity (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun requested review of this revision.Apr 24 2023, 2:21 PM
jon added inline comments.
services/identity/src/config.rs
100

If we can borrow here, I think we should. Guess it doesn't really matter

This revision is now accepted and ready to land.Apr 24 2023, 3:23 PM
services/identity/src/config.rs
100

contents is already a &str so the borrow is redundant, which clippy doesn't like

This revision was automatically updated to reflect the committed changes.