Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Jun 21, 2025

No description provided.

@rustbot
Copy link
Collaborator

rustbot commented Jun 21, 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 Jun 21, 2025
@rustbot

This comment has been minimized.

@ghost ghost force-pushed the master branch from 407e766 to 425d761 Compare June 21, 2025 19:30
@rustbot

This comment has been minimized.

@ghost ghost force-pushed the master branch from 4c24655 to ffaeef3 Compare June 22, 2025 20:24
…indows`

Suggests using (nightly) `array_chunks`, `_mut`, and `array_windows`

changelog: new lint: [`const_sized_chunks_exact`]
changelog: new lint: [`const_sized_chunks_exact_mut`]
changelog: new lint: [`const_sized_windows`]

Co-authored-by: Michał Chodzikiewicz <[email protected]>
@ghost ghost force-pushed the master branch from ffaeef3 to f8a1266 Compare June 22, 2025 20:33
@rustbot rustbot added the A-lint Area: New lints label Aug 2, 2025
@rustbot
Copy link
Collaborator

rustbot commented Sep 9, 2025

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@rustbot
Copy link
Collaborator

rustbot commented Sep 9, 2025

⚠️ Warning ⚠️

  • The following commits have merge commits (commits with multiple parents) in your changes. We have a no merge policy so these commits will need to be removed for this pull request to be merged.

    You can start a rebase with the following commands:

    $ # rebase
    $ git pull --rebase https://github.com/rust-lang/rust-clippy.git master
    $ git push --force-with-lease
    

@rustbot rustbot added has-merge-commits PR has merge commits, merge with caution. S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) labels Sep 9, 2025
@ghost ghost closed this Sep 9, 2025
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 9, 2025
@github-actions
Copy link

github-actions bot commented Sep 9, 2025

Lintcheck changes for 35bc2af

Lint Added Removed Changed
clippy::const_sized_chunks_exact 3 0 0
clippy::const_sized_windows 1 0 0

This comment will be updated if you push new changes

@ghost
Copy link
Author

ghost commented Sep 9, 2025

@rustbot label -A-lint -has-merge-commits -S-waiting-on-author

@rustbot
Copy link
Collaborator

rustbot commented Sep 9, 2025

Error: Label has-merge-commits can only be set by Rust team members

Please file an issue on GitHub at triagebot if there's a problem with this bot, or reach out on #triagebot on Zulip.

@ghost
Copy link
Author

ghost commented Sep 9, 2025

@rustbot label -A-lint -S-waiting-on-author

@rustbot rustbot removed A-lint Area: New lints S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) labels Sep 9, 2025
@ghost
Copy link
Author

ghost commented Sep 9, 2025

@rustbot release-assignment

@rustbot
Copy link
Collaborator

rustbot commented Sep 9, 2025

Assignment is not allowed on a closed PR.

@ghost
Copy link
Author

ghost commented Sep 9, 2025

@rustbot ready

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 9, 2025
@ghost
Copy link
Author

ghost commented Sep 9, 2025

@rustbot -S-waiting-on-author

@ghost
Copy link
Author

ghost commented Sep 9, 2025

@rustbot -S-waiting-on-review

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-merge-commits PR has merge commits, merge with caution. 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.

2 participants