Re: Improved find-all command now in devel

2023-01-27 Thread Edward K. Ream
On Wednesday, January 25, 2023 at 8:21:21 AM UTC-6 Edward K. Ream wrote: PR #3099 has been merged into devel. This PR: - Fixes serious bugs in the find-all command. - Improves the report in the node that find-all creates. - The find-all

Re: Improved find-all command now in devel

2023-01-25 Thread Thomas Passin
Thanks for this hard work! The new results format is certainly a big improvement! And the new code reads better, too. I would probably have tried to move make_result_from_matches() outside _find_all_helper()and let it be called separately after the matches are returned, but at least this way

Re: Improved find-all command now in devel

2023-01-25 Thread Edward K. Ream
On Wednesday, January 25, 2023 at 8:21:21 AM UTC-6 Edward K. Ream wrote: > The find-all command now returns a dictionary describing in detail what was found. A recent rev in devel updates the docstring for do_find_all as follows. QQQ Scripts can call find.do_find_all directly to get detailed

Improved find-all command now in devel

2023-01-25 Thread Edward K. Ream
PR #3099 has been merged into devel. This PR: - Fixes serious bugs in the find-all command. - Improves the report in the node that find-all creates. - The find-all command now returns a dictionary describing in detail what was found. The PR