Skip to content

Conversation

relaxcn
Copy link
Contributor

@relaxcn relaxcn commented Aug 26, 2025

Fix: #15258

changelog: suggest full path in [clone_on_ref_ptr] to avoid "failed to resolve" error.

@rustbot
Copy link
Collaborator

rustbot commented Aug 26, 2025

r? @y21

rustbot has assigned @y21.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 26, 2025
Copy link

Lintcheck changes for cfc9b8f

Lint Added Removed Changed
clippy::clone_on_ref_ptr 0 0 91

This comment will be updated if you push new changes

Copy link
Member

@y21 y21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wish we had a way to do this in a machine-applicable way, only expanding it if it isn't in scope, but oh well. This works. Thanks!

@y21 y21 added this pull request to the merge queue Aug 27, 2025
Merged via the queue into rust-lang:master with commit d727fa2 Aug 27, 2025
13 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

clone-on-ref-ptr could suggest std::sync::Arc when Arc is not in scope
3 participants