Skip to content

Remove invalid named expr test from formatter#10995

Merged
dhruvmanila merged 1 commit intodhruv/parserfrom
dhruv/remove-del-named-expr
Apr 17, 2024
Merged

Remove invalid named expr test from formatter#10995
dhruvmanila merged 1 commit intodhruv/parserfrom
dhruv/remove-del-named-expr

Conversation

@dhruvmanila
Copy link
Copy Markdown
Member

Summary

This PR removes a test case from the formatter which uses a named expression as del target. This is an invalid syntax in the new parser.

@dhruvmanila dhruvmanila added internal An internal refactor or improvement parser Related to the parser labels Apr 17, 2024
@dhruvmanila dhruvmanila force-pushed the dhruv/reset-ctx-in-parenthesized branch from d52a6ce to a4fb8c0 Compare April 17, 2024 11:51
@dhruvmanila dhruvmanila force-pushed the dhruv/remove-del-named-expr branch from d6c4c3a to d6b072d Compare April 17, 2024 11:57
Base automatically changed from dhruv/reset-ctx-in-parenthesized to dhruv/parser April 17, 2024 12:45
@dhruvmanila dhruvmanila force-pushed the dhruv/remove-del-named-expr branch from d6b072d to 8020b40 Compare April 17, 2024 12:46
@dhruvmanila dhruvmanila merged commit eaacca8 into dhruv/parser Apr 17, 2024
@dhruvmanila dhruvmanila deleted the dhruv/remove-del-named-expr branch April 17, 2024 12:46
@github-actions
Copy link
Copy Markdown
Contributor

ruff-ecosystem results

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

dhruvmanila added a commit that referenced this pull request Apr 18, 2024
## Summary

This PR removes a test case from the formatter which uses a named
expression as `del` target. This is an invalid syntax in the new parser.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal An internal refactor or improvement parser Related to the parser

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants