Skip to content

[Merged by Bors] - Remove unnecessary clippy allows#7768

Closed
rparrett wants to merge 1 commit intobevyengine:mainfrom
rparrett:try-remove-allow
Closed

[Merged by Bors] - Remove unnecessary clippy allows#7768
rparrett wants to merge 1 commit intobevyengine:mainfrom
rparrett:try-remove-allow

Conversation

@rparrett
Copy link
Contributor

Objective

I think that these allows are no longer necessary after #6534.

Solution

Let's remove them and see if clippy complains.

@rparrett rparrett changed the title Removing clippy allows Remove unnecessary clippy allows Feb 20, 2023
@alice-i-cecile alice-i-cecile added the C-Code-Quality A section of code that is hard to understand or change label Feb 20, 2023
@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Feb 20, 2023
@alice-i-cecile
Copy link
Member

Waiting to see if this passes CI.

@mockersf
Copy link
Member

Bors r+

bors bot pushed a commit that referenced this pull request Feb 20, 2023
# Objective

I think that these allows are no longer necessary after #6534.

## Solution

Let's remove them and see if clippy complains.
@bors bors bot changed the title Remove unnecessary clippy allows [Merged by Bors] - Remove unnecessary clippy allows Feb 21, 2023
@bors bors bot closed this Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-Code-Quality A section of code that is hard to understand or change S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants