Skip to content

Conversation

AA-Turner
Copy link
Member

@AA-Turner AA-Turner commented Sep 18, 2025

Addresses @serhiy-storchaka's comments following initial merge (thank you Serhiy for the reviews!).

A

@erlend-aasland
Copy link
Contributor

Deferring to Serhiy, as this is addressing his review remarks.

"""
self.expect_failure(block, err, lineno=6)

def test_double_star_after_var_keyword(self):
Copy link
Member

Choose a reason for hiding this comment

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

Should be renamed.

Copy link
Member

Choose a reason for hiding this comment

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

And, is there a test for a single ** without name?

Comment on lines -1647 to -1651
""", """
with_kwds
[
**kwds: dict
]
Copy link
Member

Choose a reason for hiding this comment

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

Could you add also a test for var-positional?

            with_varpos
                [
                *args: tuple
                ]

If it is an error and has not yet been tested.

If it is an error, and the error message is the same as for keyword arguments here, then it would be reasonable to make the error message for var-keyword the same. Otherwise we can leave a discrepancy.

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

Successfully merging this pull request may close these issues.

3 participants