Skip to content

Conversation

@Julian
Copy link
Contributor

@Julian Julian commented Jan 7, 2025

@echoix
Copy link

echoix commented Jan 9, 2025

I'm just an outsider here, (but review PR for other projects I work on), but I think this is fine, and the meaning of tool table is already used around, so definitely better with this PR. Perhaps have a second read at the title of the PR ("Minor fix to a missing WHAT? in the inline ..."), and it's good to go!

@Julian
Copy link
Contributor Author

Julian commented Jan 9, 2025

The title was intentional :) always room for a bit of fun. Thanks for the review.

@echoix
Copy link

echoix commented Jan 9, 2025

The title was intentional :) always room for a bit of fun. Thanks for the review.

Haha, I fell right into it!

@webknjaz webknjaz enabled auto-merge January 14, 2025 19:04
Copy link
Member

@webknjaz webknjaz left a comment

Choose a reason for hiding this comment

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

Thanks!

@webknjaz webknjaz added this pull request to the merge queue Jan 14, 2025
Merged via the queue into pypa:main with commit eb7d0b6 Jan 14, 2025
8 checks passed
@Julian Julian deleted the patch-2 branch January 14, 2025 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants