Skip to content

Conversation

@LikeLakers2
Copy link
Contributor

@LikeLakers2 LikeLakers2 commented Jan 14, 2025

Objective

Fixes #17111

Solution

Move #![warn(clippy::allow_attributes, clippy::allow_attributes_without_reason)] to the workspace Cargo.toml

Testing

Lots of CI testing, and local testing too.

@BenjaminBrienen BenjaminBrienen added D-Trivial Nice and easy! A great choice to get started with Bevy C-Code-Quality A section of code that is hard to understand or change A-Cross-Cutting Impacts the entire engine S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels Jan 14, 2025
Co-authored-by: Benjamin Brienen <benjamin.brienen@outlook.com>
@alice-i-cecile alice-i-cecile added this to the 0.16 milestone Jan 14, 2025
@LikeLakers2
Copy link
Contributor Author

LikeLakers2 commented Jan 14, 2025

This PR will be removed from draft mode after CI completes successfully.

@BenjaminBrienen BenjaminBrienen removed D-Trivial Nice and easy! A great choice to get started with Bevy S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels Jan 14, 2025
@BenjaminBrienen BenjaminBrienen added D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jan 14, 2025
@LikeLakers2 LikeLakers2 marked this pull request as ready for review January 14, 2025 23:21
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jan 15, 2025
@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jan 15, 2025
Merged via the queue into bevyengine:main with commit 26bb0b4 Jan 15, 2025
35 checks passed
@LikeLakers2 LikeLakers2 deleted the lint/warn_on_allow_and_without_reason/bevy branch January 15, 2025 01:37
mrchantey pushed a commit to mrchantey/bevy that referenced this pull request Feb 4, 2025
…out_reason)]` to the workspace `Cargo.toml` (bevyengine#17374)

# Objective
Fixes bevyengine#17111

## Solution
Move `#![warn(clippy::allow_attributes,
clippy::allow_attributes_without_reason)]` to the workspace `Cargo.toml`

## Testing
Lots of CI testing, and local testing too.

---------

Co-authored-by: Benjamin Brienen <benjamin.brienen@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Cross-Cutting Impacts the entire engine C-Code-Quality A section of code that is hard to understand or change D-Straightforward Simple bug fixes and API improvements, docs, test and examples 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.

Warn the clippy::allow_attributes and clippy::allow_attributes_without_reason lints

3 participants