search

ReviewNB / treon

Easy to use test framework for Jupyter Notebooks
https://reviewnb.com
MIT License
305 stars 29 forks source link

Allow multiple PATHs to be provided #24

Closed garrison closed 2 years ago

garrison commented 2 years ago

According to treon --help, treon is meant to be invoked as

treon [PATH] [--threads=<number>] [-v] [--exclude=<string>]...

However, it is currently not possible to pass multiple PATHs, unlike many command-line tools (e.g., cat). If multiple PATHs are provided, the help text is displayed and nothing else happens.

It is possible to exclude multiple paths (#1), but currently, the only way to include multiple paths (e.g., to test multiple notebooks by individual filename or to test multiple directories) is to invoke treon multiple times sequentially, once for each. But this means that the full benefits of multithreading are not available.

amit1rrr commented 2 years ago

Thanks for opening this & sending out the PR to make this improvement.

I agree, accepting multiple paths would be a good addition to treon.