Repository navigation

[ruff] Implement `incorrectly-parenthesized-tuple-in-subscript` (`RUF031`) - #12480

Merged

AlexWaygood merged 35 commits into

astral-sh:mainfrom

dylwil3:getitem-with-parens-tuple

Aug 7, 2024

Commits on Aug 7, 2024
- add test fixture for lint rule
  
  dylwil3
  authored and
  AlexWaygood
  committed
  8acd409
- add test cases with newlines
  
  dylwil3
  authored and
  AlexWaygood
  committed
  e9aa21c
- add module to check for parens around tuples in getitem calls
  
  dylwil3
  authored and
  AlexWaygood
  committed
  d59408f
- add to codes as RUF031
  
  dylwil3
  authored and
  AlexWaygood
  committed
  7882fcd
- add pointer to test case for rule
  
  dylwil3
  authored and
  AlexWaygood
  committed
  be86fe9
- implement rule for subscript expressions in checker
  
  dylwil3
  authored and
  AlexWaygood
  committed
  55c334a
- update schema
  
  dylwil3
  authored and
  AlexWaygood
  committed
  ec086e8
- add snapshot for RUF031
  
  dylwil3
  authored and
  AlexWaygood
  committed
  284a194
- reformat example
  
  dylwil3
  authored and
  AlexWaygood
  committed
  9513c9c
- begin adding settings functionality to ruff ruleset
  
  dylwil3
  authored and
  AlexWaygood
  committed
  5b0a92a
- rename rule and add config for ruff
  
  dylwil3
  authored and
  AlexWaygood
  committed
  61106a6
- update rule fix and violation logic to respect configuration
  
  dylwil3
  authored and
  AlexWaygood
  committed
  f3db34e
- update snapshot
  
  dylwil3
  authored and
  AlexWaygood
  committed
  3f5d5bb
- update schema
  
  dylwil3
  authored and
  AlexWaygood
  committed
  a0e7b04
- fix ruff settings namespace copypasta mistake and regenerate snapshot
  
  dylwil3
  authored and
  AlexWaygood
  committed
  0c339ab
- add test case with parens on setting
  
  dylwil3
  authored and
  AlexWaygood
  committed
  c2e350b
- make ruff an option group
  
  dylwil3
  authored and
  AlexWaygood
  committed
  5e1e0a4
- rename rule to satisfy convention
  
  dylwil3
  authored and
  AlexWaygood
  committed
  a696961
- change rule and config option names
  
  dylwil3
  authored and
  AlexWaygood
  committed
  209add0
- typo
  
  authored and
  AlexWaygood
  committed
  bf5947d
- Use `let .. else`
  
  authored and
  AlexWaygood
  committed
  2a895b6
- resolve comments
  
  dylwil3
  authored and
  AlexWaygood
  committed
  dedda82
- update snapshot
  
  dylwil3
  authored and
  AlexWaygood
  committed
  d8ea3a2
- treat single element tuples as well
  
  dylwil3
  authored and
  AlexWaygood
  committed
  f7ff6a3
- update snapshot
  
  dylwil3
  authored and
  AlexWaygood
  committed
  dd3d881
- clippy
  
  dylwil3
  authored and
  AlexWaygood
  committed
  cec2241
- typo and subscript instead of getitem
  
  dylwil3
  authored and
  AlexWaygood
  committed
  96fcb28
- update snapshot again
  
  dylwil3
  authored and
  AlexWaygood
  committed
  bbe1801
- change name of setting
  
  dylwil3
  authored and
  AlexWaygood
  committed
  5a83811
- update schema
  
  dylwil3
  authored and
  AlexWaygood
  committed
  4cc9eb5
- update docstrings
  
  dylwil3
  authored and
  AlexWaygood
  committed
  767ae99
- rename module and remove clippy allow
  dylwil3
  committed
  b95f068
- match to if/else
  dylwil3
  committed
  84afc6e
- Apply suggestions from code review
  AlexWaygood
  authored
  ccf47a7
- final nits
  AlexWaygood
  committed
  4ba055d

Navigation Menu

Repository navigation

[ruff] Implement `incorrectly-parenthesized-tuple-in-subscript` (`RUF031`) - #12480

[ruff] Implement `incorrectly-parenthesized-tuple-in-subscript` (`RUF031`)#12480
AlexWaygood merged 35 commits into
astral-sh:mainfrom
dylwil3:getitem-with-parens-tuple

Commits on Aug 7, 2024

add test fixture for lint rule

add test cases with newlines

add module to check for parens around tuples in getitem calls

add to codes as RUF031

add pointer to test case for rule

implement rule for subscript expressions in checker

update schema

add snapshot for RUF031

reformat example

begin adding settings functionality to ruff ruleset

rename rule and add config for ruff

update rule fix and violation logic to respect configuration

update snapshot

update schema

fix ruff settings namespace copypasta mistake and regenerate snapshot

add test case with parens on setting

make ruff an option group

rename rule to satisfy convention

change rule and config option names

typo

Use `let .. else`

resolve comments

update snapshot

treat single element tuples as well

update snapshot

clippy

typo and subscript instead of getitem

update snapshot again

change name of setting

update schema

update docstrings

rename module and remove clippy allow

match to if/else

Apply suggestions from code review

final nits