Skip to content

Conversation

@cyyever
Copy link
Contributor

@cyyever cyyever commented Oct 22, 2025

What does this PR do?

This PR enables all flake8-pie rules in ruff. These rules are:

PIE790   Unnecessary pass statement
PIE794   Class field {name} is defined multiple times 
PIE796   Enum contains duplicate value: {value}  
PIE800   Unnecessary spread ** 
PIE804   Unnecessary dict kwargs  
PIE807   Prefer {container} over useless lambda 
PIE808   Unnecessary start argument in range  
PIE810   Call {attr} once with a tuple   

@github-actions
Copy link
Contributor

[For maintainers] Suggested jobs to run (before merge)

run-slow: bert, bertweet, big_bird, clap, conditional_detr, detr, funnel, glm4v, glm4v_moe, gpt_neo, gpt_sw3, instructblipvideo, internvl, luke, marian, maskformer

Signed-off-by: Yuanyuan Chen <[email protected]>
@Rocketknight1
Copy link
Member

Adding more ruff rules so cc core maintainer @ArthurZucker

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants