Page MenuHomePhabricator

[identity] some nits
ClosedPublic

Authored by varun on Apr 24 2023, 1:51 PM.
Tags
None
Referenced Files
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
Unknown Object (File)
Tue, Mar 12, 3:47 PM
Unknown Object (File)
Thu, Mar 7, 12:54 AM
Unknown Object (File)
Nov 7 2023, 4:04 AM
Unknown Object (File)
Oct 16 2023, 10:14 AM
Subscribers

Details

Summary

small changes to satisfy cargo clippy

Depends on D7595

Test Plan

cargo build

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun requested review of this revision.Apr 24 2023, 2:21 PM
jon added inline comments.
services/identity/src/config.rs
100 ↗(On Diff #25628)

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 ↗(On Diff #25628)

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.