ruff
Consider single element subscript expr for implicit optional
#5717
Merged
Overview
Commits
14
Changes
View On GitHub

Consider single element subscript expr for implicit optional #5717

dhruvmanila merged 14 commits into main from dhruv/implicit-optional-tests
dhruvmanila
dhruvmanila
dhruvmanila dhruvmanila marked this pull request as draft 2 years ago
github-actions
dhruvmanila dhruvmanila force pushed from 8955ef98 to e47034a2 2 years ago
dhruvmanila dhruvmanila changed the title Update implicit optional tests for coverage Consider single element subscript expr for implicit optional 2 years ago
dhruvmanila dhruvmanila marked this pull request as ready for review 2 years ago
charliermarsh
charliermarsh commented on 2023-07-12
dhruvmanila Move `TypingTarget` related logic out of rule file
f8e5accf
dhruvmanila Check if `TypingTarget` can be resolved to `Any`
1553d2ac
dhruvmanila Add `Unknown` type to keep `Any` separate
53f4a2a8
dhruvmanila Tests
713d5007
dhruvmanila Explicit `Known` type, avoid flagging in case of failures
182def17
dhruvmanila Wrap the only expr in the vector instead of panic
b082d103
dhruvmanila Remove bugfix code (results in false negatives)
efbfc9c5
dhruvmanila Temporarily disable `ANN401` in scripts
01866f94
dhruvmanila Update implicit optional tests for coverage
aa5d0d16
dhruvmanila Account for one element subscript inside typing
f79d1b14
dhruvmanila Increase max iteration in tests
9e8416ea
dhruvmanila Avoid allocation while resolving slice value
a01c378f
dhruvmanila Revert "Temporarily disable `ANN401` in scripts"
bd27d7ad
dhruvmanila dhruvmanila force pushed from 87f7fc27 to bd27d7ad 2 years ago
dhruvmanila dhruvmanila requested a review from charliermarsh charliermarsh 2 years ago
dhruvmanila
dhruvmanila commented on 2023-07-13
Base automatically changed from dhruv/typing-target-contains-any to main 2 years ago
dhruvmanila Merge branch 'main' into dhruv/implicit-optional-tests
70d5749a
dhruvmanila dhruvmanila enabled auto-merge (squash) 2 years ago
dhruvmanila dhruvmanila added rule
dhruvmanila dhruvmanila merged cf48ad7b into main 2 years ago
dhruvmanila dhruvmanila deleted the dhruv/implicit-optional-tests branch 2 years ago

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
rule
Milestone
No milestone
Terms Of Use Privacy Policy Refund Policy Impressum