Page MenuHomePhabricator
Differential D7596

[identity] some nits
ClosedPublic
Actions

Authored by varun on Apr 24 2023, 1:51 PM.
Tags
None
Referenced Files
F1690473: D7596.id25644.diff
Wed, May 1, 9:46 PM
Unknown Object (File)
Tue, Apr 30, 3:58 PM
Unknown Object (File)
Sun, Apr 28, 5:36 PM
Unknown Object (File)
Sun, Apr 28, 5:25 AM
Unknown Object (File)
Wed, Apr 24, 5:22 PM
Unknown Object (File)
Mar 28 2024, 7:41 PM
Unknown Object (File)
Mar 28 2024, 7:41 PM
Unknown Object (File)
Mar 28 2024, 7:41 PM
View All 24 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
Branch
identity (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

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

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

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 25628

View Options

services/identity/src/config.rs

Loading...