Skip to content

Conversation

@felixxm
Copy link
Contributor

@felixxm felixxm commented Mar 27, 2024

Thanks Chris Jerdonek for the implementation idea.
@ziima
Copy link
Contributor

ziima commented Jul 13, 2024

The new method is not documented, otherwise this looks good to me.

@felixxm
Copy link
Contributor Author

felixxm commented Jul 14, 2024

The new method is not documented, otherwise this looks good to me.

@ziima Thanks for the review 👍 I documented getName() and moved release notes to 3.14.

Copy link
Contributor

@adamchainz adamchainz left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@MarkusH MarkusH left a comment

Choose a reason for hiding this comment

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

I can see a benefit of having this for e.g. @adamchainz's unittest-parametrize package.

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.

4 participants