Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add information about qt_viewer fixture #505

Merged
merged 2 commits into from
Oct 28, 2024

Conversation

Czaki
Copy link
Contributor

@Czaki Czaki commented Oct 25, 2024

References and relevant issues

Try to address napari/napari#7251 (comment)

Description

Add information about qt_viewer fixture and try to clarify when to use it.

Copy link
Member

@melissawm melissawm left a comment

Choose a reason for hiding this comment

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

Left a couple of wording suggestions, take them or leave them! Thanks @Czaki

docs/developers/contributing/testing.md Outdated Show resolved Hide resolved
docs/developers/contributing/testing.md Outdated Show resolved Hide resolved
docs/developers/contributing/testing.md Outdated Show resolved Hide resolved
Co-authored-by: Melissa Weber Mendonça <melissawm@gmail.com>
Copy link
Member

@psobolewskiPhD psobolewskiPhD left a comment

Choose a reason for hiding this comment

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

This is really nice, thanks to both of you!
❤️

@psobolewskiPhD psobolewskiPhD added this to the 0.5.5 milestone Oct 26, 2024
@melissawm melissawm merged commit b9e26a3 into napari:main Oct 28, 2024
8 checks passed
@Czaki Czaki deleted the qt_viewer_fixture_docs branch October 28, 2024 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants