New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
clippy: Fix warnings in components/script/dom/request.rs
#31867
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! I only have one nit request for these changes. Let's avoid abbreviating in this case since "value" is only two characters longer and increases clarity.
components/script/dom/request.rs
Outdated
impl From<RequestCache> for NetTraitsRequestCache { | ||
fn from(val: RequestCache) -> Self { | ||
match val { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
impl From<RequestCache> for NetTraitsRequestCache { | |
fn from(val: RequestCache) -> Self { | |
match val { | |
impl From<RequestCache> for NetTraitsRequestCache { | |
fn from(value: RequestCache) -> Self { | |
match value { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've gone ahead and made these changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies, I was preoccupied with some personal work and couldn't attend to the changes requested.
Thank you for making the commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No worries. Thank you for the contribution!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies. I didn't set the proper review flag before.
Major changes are:
./mach build -d
does not report any errors./mach test-tidy
does not report any errors