Skip to content

Conversation

@samcunliffe
Copy link
Collaborator

Copy link
Member

@dstansby dstansby left a comment

Choose a reason for hiding this comment

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

Looks good overall - I spotted one typo, hence request changes, but the rest looks good 👍

pyproject.toml Outdated
"tox",
"pytest-xvfb; sys_platform == 'linux'",
]
dev = [
Copy link
Member

Choose a reason for hiding this comment

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

Could this instead be ["napari-matplotlib[test]", "napar-matplotlib[docs]"] to avoid duplication?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, you can have that!

The syntax is a bit different. It would be:

dev = [
   {include-group = "test"},
   {include-group = "docs"},
]

but agree that we don't want to literally WET.

Co-authored-by: David Stansby <dstansby@gmail.com>
@samcunliffe samcunliffe marked this pull request as ready for review February 2, 2026 11:27
@samcunliffe samcunliffe requested a review from dstansby February 2, 2026 11:28
@samcunliffe samcunliffe self-assigned this Feb 2, 2026
@samcunliffe samcunliffe added the Maintenance Not related to the development of new features label Feb 2, 2026
@dstansby dstansby merged commit 5b2a97c into matplotlib:main Feb 2, 2026
12 checks passed
@samcunliffe
Copy link
Collaborator Author

@jaideepkathiresan thanks for the contribution!

@samcunliffe samcunliffe deleted the tox-ini-to-pyproject branch February 2, 2026 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintenance Not related to the development of new features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants