Skip to content

[IMPROVEMENT] Update ruff rules YTT, BLE, RERF, Q, EXE, FA, FURB, FLY, SLOT, NPY, W, TID, PIE, RSE, TRY002#199

Merged
Thijss merged 3 commits intoPowerGridModel:mainfrom
furqan463:ruff_rule
Mar 23, 2026
Merged

[IMPROVEMENT] Update ruff rules YTT, BLE, RERF, Q, EXE, FA, FURB, FLY, SLOT, NPY, W, TID, PIE, RSE, TRY002#199
Thijss merged 3 commits intoPowerGridModel:mainfrom
furqan463:ruff_rule

Conversation

@furqan463
Copy link
Contributor

Relates Issue #188

Changes proposed in this PR include:

Added ruff rules requiring little to no change in code.
Added ruff rules: YTT, BLE, RERF, Q, EXE, FA, FURB, FLY, SLOT, NPY, W, TID, PIE, RSE, TRY002

Added all other desired ruff rules in pyproject.toml but commented out.

Signed-off-by: furqan463 <ahmadfurqanc@gmail.com>
Signed-off-by: furqan463 <ahmadfurqanc@gmail.com>
Copy link
Member

@Thijss Thijss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍

@Thijss Thijss merged commit a887843 into PowerGridModel:main Mar 23, 2026
13 checks passed
@furqan463 furqan463 deleted the ruff_rule branch March 23, 2026 10:34
@furqan463 furqan463 changed the title [IMPROVEMENT] Update ruff rules [IMPROVEMENT] Update ruff rules YTT, BLE, RERF, Q, EXE, FA, FURB, FLY, SLOT, NPY, W, TID, PIE, RSE, TRY002 Mar 23, 2026
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