karuppuchamysuresh opened a new pull request, #20058:
URL: https://github.com/apache/datafusion/pull/20058
Enable the `clippy::clone_on_ref_ptr` lint at the workspace level to ensure
consistent enforcement across all crates. This makes Arc clones explicit
throughout the codebase.
Changes:
- Added [workspace.lints.clippy] section to root Cargo.toml
- Added [lints] workspace = true to subcrates that were missing it
- Removed individual #![deny(clippy::clone_on_ref_ptr)] declarations from
all lib.rs and mod.rs files
Fixes #17083
## Which issue does this PR close?
Standardize the `clippy::clone_on_ref_ptr` lint configuration across the
entire
DataFusion workspace. This ensures Arc clones are explicit throughout the
codebase
and makes the linting rules more maintainable.
## What changes are included in this PR?
- Added `[workspace.lints.clippy]` section to root Cargo.toml with
`clone_on_ref_ptr = "deny"`
- Added `[lints] workspace = true` to subcrates that were missing it:
- datafusion/proto/Cargo.toml
- datafusion/proto-common/Cargo.toml
- datafusion/physical-expr-adapter/Cargo.toml
- Removed individual `#![deny(clippy::clone_on_ref_ptr)]` declarations from
34 source files
## Are these changes tested?
The lint configuration is enforced at compile time. CI will validate that:
1. The workspace lint configuration is properly inherited by all crates
2. All code complies with the lint rules
## Are there any user-facing changes?
No
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
