From 447108b2a44f78ea2020b0b184c0189a70dc7bd8 Mon Sep 17 00:00:00 2001 From: MichiRecRoom <1008889+LikeLakers2@users.noreply.github.com> Date: Sun, 12 Jan 2025 00:28:26 -0500 Subject: [PATCH] Downgrade `clippy::allow_attributes` and `clippy::allow_attributes_without_reason` to `warn` (#17320) # Objective I realized that setting these to `deny` may have been a little aggressive - especially since we upgrade warnings to denies in CI. ## Solution Downgrades these lints to `warn`, so that compiles can work locally. CI will still treat these as denies. --- crates/bevy_a11y/src/lib.rs | 2 +- crates/bevy_animation/src/lib.rs | 2 +- crates/bevy_app/src/lib.rs | 2 +- crates/bevy_asset/src/lib.rs | 2 +- crates/bevy_audio/src/lib.rs | 2 +- crates/bevy_color/src/lib.rs | 2 +- crates/bevy_dev_tools/src/lib.rs | 2 +- crates/bevy_diagnostic/src/lib.rs | 2 +- crates/bevy_gilrs/src/lib.rs | 2 +- crates/bevy_gizmos/src/lib.rs | 2 +- crates/bevy_gltf/src/lib.rs | 2 +- crates/bevy_hierarchy/src/lib.rs | 2 +- crates/bevy_image/src/lib.rs | 2 +- crates/bevy_input/src/lib.rs | 2 +- crates/bevy_internal/src/lib.rs | 2 +- crates/bevy_log/src/lib.rs | 2 +- crates/bevy_math/src/lib.rs | 2 +- crates/bevy_mesh/src/lib.rs | 2 +- crates/bevy_pbr/src/lib.rs | 2 +- crates/bevy_picking/src/lib.rs | 2 +- crates/bevy_ptr/src/lib.rs | 2 +- crates/bevy_reflect/src/lib.rs | 2 +- crates/bevy_remote/src/lib.rs | 2 +- crates/bevy_render/src/lib.rs | 2 +- crates/bevy_scene/src/lib.rs | 2 +- crates/bevy_sprite/src/lib.rs | 2 +- crates/bevy_state/src/lib.rs | 2 +- crates/bevy_tasks/src/lib.rs | 2 +- crates/bevy_text/src/lib.rs | 2 +- crates/bevy_time/src/lib.rs | 2 +- crates/bevy_transform/src/lib.rs | 2 +- crates/bevy_ui/src/lib.rs | 2 +- crates/bevy_utils/src/lib.rs | 2 +- crates/bevy_window/src/lib.rs | 2 +- crates/bevy_winit/src/lib.rs | 2 +- 35 files changed, 35 insertions(+), 35 deletions(-) diff --git a/crates/bevy_a11y/src/lib.rs b/crates/bevy_a11y/src/lib.rs index a0759303e023b..9a7924998701c 100644 --- a/crates/bevy_a11y/src/lib.rs +++ b/crates/bevy_a11y/src/lib.rs @@ -1,5 +1,5 @@ #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_animation/src/lib.rs b/crates/bevy_animation/src/lib.rs index 25686b4916ad6..72b898b3e054e 100644 --- a/crates/bevy_animation/src/lib.rs +++ b/crates/bevy_animation/src/lib.rs @@ -1,6 +1,6 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_app/src/lib.rs b/crates/bevy_app/src/lib.rs index ae0af19c5115f..5523f0ceaea54 100644 --- a/crates/bevy_app/src/lib.rs +++ b/crates/bevy_app/src/lib.rs @@ -7,7 +7,7 @@ )] #![cfg_attr(any(docsrs, docsrs_dep), feature(doc_auto_cfg, rustdoc_internals))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_asset/src/lib.rs b/crates/bevy_asset/src/lib.rs index 3d58b701b0c87..ec5c7905a3abe 100644 --- a/crates/bevy_asset/src/lib.rs +++ b/crates/bevy_asset/src/lib.rs @@ -139,7 +139,7 @@ //! This trait mirrors [`AssetLoader`] in structure, and works in tandem with [`AssetWriter`](io::AssetWriter), which mirrors [`AssetReader`](io::AssetReader). #![expect(missing_docs, reason = "Not all docs are written yet, see #3492.")] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_audio/src/lib.rs b/crates/bevy_audio/src/lib.rs index c5b777c442be1..5734d68c10abd 100644 --- a/crates/bevy_audio/src/lib.rs +++ b/crates/bevy_audio/src/lib.rs @@ -1,5 +1,5 @@ #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_color/src/lib.rs b/crates/bevy_color/src/lib.rs index cf13c86360062..dfdcb0e666a3b 100644 --- a/crates/bevy_color/src/lib.rs +++ b/crates/bevy_color/src/lib.rs @@ -1,6 +1,6 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_dev_tools/src/lib.rs b/crates/bevy_dev_tools/src/lib.rs index 40f8ce58f5699..cf9eed7f75d17 100644 --- a/crates/bevy_dev_tools/src/lib.rs +++ b/crates/bevy_dev_tools/src/lib.rs @@ -1,6 +1,6 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_diagnostic/src/lib.rs b/crates/bevy_diagnostic/src/lib.rs index cecc64f777e8f..e003a0093866c 100644 --- a/crates/bevy_diagnostic/src/lib.rs +++ b/crates/bevy_diagnostic/src/lib.rs @@ -1,7 +1,7 @@ #![expect(missing_docs, reason = "Not all docs are written yet, see #3492.")] #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_gilrs/src/lib.rs b/crates/bevy_gilrs/src/lib.rs index 7b6881a3ed1bf..c9ffa983ca25b 100644 --- a/crates/bevy_gilrs/src/lib.rs +++ b/crates/bevy_gilrs/src/lib.rs @@ -1,6 +1,6 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_gizmos/src/lib.rs b/crates/bevy_gizmos/src/lib.rs index 82ae2c6eed5fd..85ab68b649d0d 100644 --- a/crates/bevy_gizmos/src/lib.rs +++ b/crates/bevy_gizmos/src/lib.rs @@ -1,5 +1,5 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_gltf/src/lib.rs b/crates/bevy_gltf/src/lib.rs index fe9a1d87e278b..e0f126af550b3 100644 --- a/crates/bevy_gltf/src/lib.rs +++ b/crates/bevy_gltf/src/lib.rs @@ -1,6 +1,6 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_hierarchy/src/lib.rs b/crates/bevy_hierarchy/src/lib.rs index eb5a2844f87d4..97e930070b37b 100644 --- a/crates/bevy_hierarchy/src/lib.rs +++ b/crates/bevy_hierarchy/src/lib.rs @@ -1,6 +1,6 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_image/src/lib.rs b/crates/bevy_image/src/lib.rs index 0137ccbe3bba4..9b48d7b60c4b6 100644 --- a/crates/bevy_image/src/lib.rs +++ b/crates/bevy_image/src/lib.rs @@ -1,5 +1,5 @@ #![expect(missing_docs, reason = "Not all docs are written yet, see #3492.")] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_input/src/lib.rs b/crates/bevy_input/src/lib.rs index ffcef2e2cb54a..11475d4fdc059 100644 --- a/crates/bevy_input/src/lib.rs +++ b/crates/bevy_input/src/lib.rs @@ -1,6 +1,6 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_internal/src/lib.rs b/crates/bevy_internal/src/lib.rs index f164173561584..a004d28d783ce 100644 --- a/crates/bevy_internal/src/lib.rs +++ b/crates/bevy_internal/src/lib.rs @@ -1,6 +1,6 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_log/src/lib.rs b/crates/bevy_log/src/lib.rs index b6b43dfdc0d4a..cd7db4556c6fc 100644 --- a/crates/bevy_log/src/lib.rs +++ b/crates/bevy_log/src/lib.rs @@ -1,5 +1,5 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_math/src/lib.rs b/crates/bevy_math/src/lib.rs index 8c758154ef9c2..fef32490f3549 100644 --- a/crates/bevy_math/src/lib.rs +++ b/crates/bevy_math/src/lib.rs @@ -1,5 +1,5 @@ #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_mesh/src/lib.rs b/crates/bevy_mesh/src/lib.rs index cc8d4c4ea31a0..0baa1db8ecc10 100644 --- a/crates/bevy_mesh/src/lib.rs +++ b/crates/bevy_mesh/src/lib.rs @@ -1,5 +1,5 @@ #![expect(missing_docs, reason = "Not all docs are written yet, see #3492.")] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_pbr/src/lib.rs b/crates/bevy_pbr/src/lib.rs index ec67b70c8b501..e3eb0f43ffca1 100644 --- a/crates/bevy_pbr/src/lib.rs +++ b/crates/bevy_pbr/src/lib.rs @@ -1,7 +1,7 @@ #![expect(missing_docs, reason = "Not all docs are written yet, see #3492.")] #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_picking/src/lib.rs b/crates/bevy_picking/src/lib.rs index ccf89cd3cbb74..086aac8f24c53 100644 --- a/crates/bevy_picking/src/lib.rs +++ b/crates/bevy_picking/src/lib.rs @@ -155,7 +155,7 @@ //! the plugin with arbitrary backends and input methods, yet still use all the high level features. #![deny(missing_docs)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_ptr/src/lib.rs b/crates/bevy_ptr/src/lib.rs index 7a0f143483dc5..9c8bd69144596 100644 --- a/crates/bevy_ptr/src/lib.rs +++ b/crates/bevy_ptr/src/lib.rs @@ -2,7 +2,7 @@ #![no_std] #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![expect(unsafe_code, reason = "Raw pointers are inherently unsafe.")] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_reflect/src/lib.rs b/crates/bevy_reflect/src/lib.rs index ea0dd188abc0c..8082533119fd1 100644 --- a/crates/bevy_reflect/src/lib.rs +++ b/crates/bevy_reflect/src/lib.rs @@ -1,5 +1,5 @@ #![expect(missing_docs, reason = "Not all docs are written yet, see #3492.")] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_remote/src/lib.rs b/crates/bevy_remote/src/lib.rs index 91370063c0ce9..772c2dd7362f9 100644 --- a/crates/bevy_remote/src/lib.rs +++ b/crates/bevy_remote/src/lib.rs @@ -305,7 +305,7 @@ //! [fully-qualified type names]: bevy_reflect::TypePath::type_path //! [fully-qualified type name]: bevy_reflect::TypePath::type_path -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_render/src/lib.rs b/crates/bevy_render/src/lib.rs index 1b7f3a76aa926..015bf8d4d8475 100644 --- a/crates/bevy_render/src/lib.rs +++ b/crates/bevy_render/src/lib.rs @@ -1,6 +1,6 @@ #![expect(missing_docs, reason = "Not all docs are written yet, see #3492.")] #![expect(unsafe_code, reason = "Unsafe code is used to improve performance.")] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_scene/src/lib.rs b/crates/bevy_scene/src/lib.rs index 161cb14947472..035fcbe0dd381 100644 --- a/crates/bevy_scene/src/lib.rs +++ b/crates/bevy_scene/src/lib.rs @@ -1,6 +1,6 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_sprite/src/lib.rs b/crates/bevy_sprite/src/lib.rs index fc6c4b5d5dfe9..cb93157069a7d 100644 --- a/crates/bevy_sprite/src/lib.rs +++ b/crates/bevy_sprite/src/lib.rs @@ -1,7 +1,7 @@ #![expect(missing_docs, reason = "Not all docs are written yet, see #3492.")] #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_state/src/lib.rs b/crates/bevy_state/src/lib.rs index e79c9008cff4b..5d3537a64c020 100644 --- a/crates/bevy_state/src/lib.rs +++ b/crates/bevy_state/src/lib.rs @@ -37,7 +37,7 @@ ) )] #![cfg_attr(any(docsrs, docsrs_dep), feature(rustdoc_internals))] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_tasks/src/lib.rs b/crates/bevy_tasks/src/lib.rs index 7b1438d4b2796..3c4f5551ad53a 100644 --- a/crates/bevy_tasks/src/lib.rs +++ b/crates/bevy_tasks/src/lib.rs @@ -4,7 +4,7 @@ html_logo_url = "https://bevyengine.org/assets/icon.png", html_favicon_url = "https://bevyengine.org/assets/icon.png" )] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_text/src/lib.rs b/crates/bevy_text/src/lib.rs index cfbd92548d28b..a1902f5b114f6 100644 --- a/crates/bevy_text/src/lib.rs +++ b/crates/bevy_text/src/lib.rs @@ -28,7 +28,7 @@ //! retrieving glyphs from the cache, or rasterizing to a [`FontAtlas`] if necessary. //! 3. [`PositionedGlyph`]s are stored in a [`TextLayoutInfo`], //! which contains all the information that downstream systems need for rendering. -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_time/src/lib.rs b/crates/bevy_time/src/lib.rs index cd6f6d298ac78..74d42026b1123 100644 --- a/crates/bevy_time/src/lib.rs +++ b/crates/bevy_time/src/lib.rs @@ -1,7 +1,7 @@ #![doc = include_str!("../README.md")] #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_transform/src/lib.rs b/crates/bevy_transform/src/lib.rs index c6de0843a0627..81b2388750ca1 100644 --- a/crates/bevy_transform/src/lib.rs +++ b/crates/bevy_transform/src/lib.rs @@ -1,6 +1,6 @@ #![doc = include_str!("../README.md")] #![cfg_attr(docsrs, feature(doc_auto_cfg))] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_ui/src/lib.rs b/crates/bevy_ui/src/lib.rs index 49ca8ab54febe..e9e0e9c64e26b 100644 --- a/crates/bevy_ui/src/lib.rs +++ b/crates/bevy_ui/src/lib.rs @@ -1,5 +1,5 @@ #![expect(missing_docs, reason = "Not all docs are written yet, see #3492.")] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_utils/src/lib.rs b/crates/bevy_utils/src/lib.rs index 48de586fe5913..f539ffc4fd1c6 100644 --- a/crates/bevy_utils/src/lib.rs +++ b/crates/bevy_utils/src/lib.rs @@ -1,4 +1,4 @@ -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_window/src/lib.rs b/crates/bevy_window/src/lib.rs index 246d7487388f5..3b6822a7c285f 100644 --- a/crates/bevy_window/src/lib.rs +++ b/crates/bevy_window/src/lib.rs @@ -1,5 +1,5 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes" diff --git a/crates/bevy_winit/src/lib.rs b/crates/bevy_winit/src/lib.rs index 5b95ed7f894c1..cac00ecbf7712 100644 --- a/crates/bevy_winit/src/lib.rs +++ b/crates/bevy_winit/src/lib.rs @@ -1,6 +1,6 @@ #![cfg_attr(docsrs, feature(doc_auto_cfg))] #![forbid(unsafe_code)] -#![deny( +#![warn( clippy::allow_attributes, clippy::allow_attributes_without_reason, reason = "See #17111; To be removed once all crates are in-line with these attributes"