Page MenuHomePhabricator
Differential D7596

[identity] some nits
ClosedPublic
Actions

Authored by varun on Apr 24 2023, 1:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 28, 7:41 PM
Unknown Object (File)
Thu, Mar 28, 7:41 PM
Unknown Object (File)
Thu, Mar 28, 7:41 PM
Unknown Object (File)
Thu, Mar 28, 7:38 PM
Unknown Object (File)
Thu, Mar 28, 5:42 AM
Unknown Object (File)
Mar 17 2024, 11:46 PM
Unknown Object (File)
Mar 12 2024, 6:14 PM
Unknown Object (File)
Mar 12 2024, 3:47 PM
View All 19 Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
• jon
bartek
Commits
rCOMMe6f7c4f4af56: [identity] some nits
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 created this revision.Apr 24 2023, 1:51 PM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptApr 24 2023, 1:51 PM
Harbormaster completed remote builds in B18715: Diff 25628.Apr 24 2023, 2:21 PM
varun requested review of this revision.Apr 24 2023, 2:21 PM
• jon accepted this revision.Apr 24 2023, 3:23 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
varun added inline comments.Apr 24 2023, 8:32 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

bartek accepted this revision.Apr 24 2023, 8:34 PM
Closed by commit rCOMMe6f7c4f4af56: [identity] some nits (authored by varun). · Explain WhyApr 24 2023, 8:34 PM
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMMe6f7c4f4af56: [identity] some nits.

Revision Contents
Changeset List

PathSize
services/
identity/
src/
6 lines

Diff 25644

View Options

services/identity/src/config.rs

Loading...