Skip to content

Conversation

alex-700
Copy link
Contributor

fixes #6502

changelog: lint "&PathBuf instead of &Path" in PTR_ARG

@rust-highfive
Copy link

r? @Manishearth

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 25, 2020
- extract get_only_generic_arg_snippet to improve readability
@alex-700 alex-700 force-pushed the add-path-buf-to-ptr-arg-lint branch from 4fddb15 to dfaea9c Compare December 25, 2020 12:06
@Manishearth
Copy link
Member

@bors r+

thanks!

@bors
Copy link
Contributor

bors commented Dec 27, 2020

📌 Commit dfaea9c has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Dec 27, 2020

⌛ Testing commit dfaea9c with merge 61a3ee7...

@bors
Copy link
Contributor

bors commented Dec 27, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Manishearth
Pushing 61a3ee7 to master...

@bors bors merged commit 61a3ee7 into rust-lang:master Dec 27, 2020
@alex-700 alex-700 deleted the add-path-buf-to-ptr-arg-lint branch December 29, 2020 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ptr_arg lint could warn also with &PathBuf and &Path
4 participants