Skip to content

Enable clippy lint clippy::allow_attributes #10458

Closed

Description

@Jefffrey

Same as we did for datafusion: apache/datafusion#18881

This should not be done in one big PR, but preferably by crates (or chunks of crates) until we can enable it at the workspace level

Also fix this line in doc once this is done:

Search for `allow(clippy::` in the codebase to identify lints that are ignored/allowed. We currently prefer ignoring lints on the lowest unit possible.

Activity

  1. Hashim1999164 commented on Jul 29, 2026

    @Hashim1999164
    Contributor

    I’ll work on enabling this lint for the arrow-string crate.

  2. added a commit that references this issue on Aug 4, 2026
  3. added a commit that references this issue on Aug 7, 2026
  4. ZhangStudyLife commented on Aug 8, 2026

    @ZhangStudyLife
    Contributor

    I'll work on enabling clippy::allow_attributes for the arrow-arith crate.

  5. WaterWhisperer commented on Aug 9, 2026

    @WaterWhisperer
    Contributor

    I'll work on enabling clippy::allow_attributes for the parquet-geospatial crate

  6. yongster commented on Aug 9, 2026

    @yongster
    Contributor

    I'd like to take the parquet-variant crate portion. I plan to enable clippy::allow_attributes for that crate and replace its single allow(unused) with an appropriately scoped expect.

  7. cakeni commented on Aug 9, 2026

    @cakeni
    Contributor

    I will take the arrow-data crate portion. I plan to enable clippy::allow_attributes for that crate, replace its single active Clippy allow with an expect, and run the crate-specific Clippy, tests, and formatting checks.

  8. 22 remaining items

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions