Skip to content
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

make sure the msrv for const_raw_ptr_deref is met when linting [missing_const_for_fn] #12713

Merged
merged 1 commit into from May 15, 2024

Conversation

J-ZhengLi
Copy link
Member

fixes: #8864


changelog: make sure the msrv for const_ptr_deref is met when linting [missing_const_for_fn]

@rustbot
Copy link
Collaborator

rustbot commented Apr 25, 2024

r? @dswij

rustbot has assigned @dswij.
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 Apr 25, 2024
@J-ZhengLi J-ZhengLi changed the title make sure the msrv for const_ptr_deref is met when linting [missing_const_for_fn] make sure the msrv for const_raw_ptr_deref is met when linting [missing_const_for_fn] Apr 25, 2024
@dswij dswij added S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels May 4, 2024
@J-ZhengLi J-ZhengLi force-pushed the issue8864 branch 3 times, most recently from f6b2d85 to e062224 Compare May 8, 2024 07:45
@J-ZhengLi
Copy link
Member Author

Finally got some time to work on this. @rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties and removed S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) labels May 8, 2024
@y21
Copy link
Member

y21 commented May 14, 2024

Since dswij seems to be a little busy and has a bunch of other PRs assigned and I already started reviewing this, I'm gonna steal this one to move it forwards. Hope you dont mind :D
r? @y21

@rustbot rustbot assigned y21 and unassigned dswij May 14, 2024
@dswij
Copy link
Member

dswij commented May 15, 2024

Yeah, feel free :)

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.

LGTM

@y21
Copy link
Member

y21 commented May 15, 2024

Thanks! @bors r+

@bors
Copy link
Collaborator

bors commented May 15, 2024

📌 Commit 1b7fc5f has been approved by y21

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented May 15, 2024

⌛ Testing commit 1b7fc5f with merge caad063...

@bors
Copy link
Collaborator

bors commented May 15, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: y21
Pushing caad063 to master...

1 similar comment
@bors
Copy link
Collaborator

bors commented May 15, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: y21
Pushing caad063 to master...

@bors bors merged commit caad063 into rust-lang:master May 15, 2024
5 checks passed
@bors
Copy link
Collaborator

bors commented May 15, 2024

👀 Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.

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.

missing-const-for-fn when const fn would require unstable features
5 participants