Page MenuHomePhabricator

[identity] some nits
ClosedPublic

Authored by varun on Apr 24 2023, 1:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 10, 12:01 AM
Unknown Object (File)
Mon, Feb 24, 9:55 AM
Unknown Object (File)
Feb 18 2025, 9:16 AM
Unknown Object (File)
Feb 14 2025, 3:42 PM
Unknown Object (File)
Feb 3 2025, 9:29 AM
Unknown Object (File)
Feb 1 2025, 1:33 AM
Unknown Object (File)
Jan 31 2025, 6:14 AM
Unknown Object (File)
Jan 31 2025, 6: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.