Conversation
Potential fix for #165 Signed-off-by: Gary O'Neall <gary@sourceauditor.com>
|
Reviewing now. Thanks for turning this around so fast @goneall! |
|
Ok so running my default (smaller) test suite here took almost 10 minutes with v1.1.10, and only 35 seconds with the All of my tests passed in each case, so functionally it appears to be working well too. tl;dr: this is a MASSIVE performance improvement!! 🎉 |
|
@pmonks - Thanks for the review. Wow! More of an improvement than I thought. Thanks for the design suggestions. I'll go ahead and merge it in. |
Potential fix for #165
It seems to speed things up - not quite fast enough to get rid of the "slow tests" flags, but a noticeable improvement.
@pmonks - review / give it a try and let me know how it does. This implements the algorithm we were discussing.