Detail a list of available pytest plugins to perform parallel exploration #108
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lysnikolaou
reviewed
Nov 5, 2024
There was a problem hiding this comment.
Thanks @andfoy! Generally looks good to me, with just a couple of things to fix.
rgommers
reviewed
Nov 5, 2024
There was a problem hiding this comment.
I think the section on pytest plugins should come first, since that's the better way to get started. As written now, doing the same thing manually is probably only useful to:
- test only a small subset of functionality with concurrent execution,
- being able to run some concurrent tests without having to add a new test dependency
Anything else? It would be good to explain why that large and complex code block is shown (i.e., the reasons above), and if the rationale for it is quite thin then consider hiding it in a foldout or in a note/info block where it gets rendered in smaller font.
ngoldbaum
reviewed
Nov 5, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See #86 (comment)