Harry Sarson
Harry Sarson
Just to warn you @jedwards1211, I am not sure this repo is actively maintained at the moment. I might be wrong but I haven't seen any activity here in a...
Thanks :) The final commit is very dubious - there may well be a better solution but I couldn't find one
@patrick-steele-idem Node v0 is not maintained anymore. Dropping support for it is probably the best solution for this.
@mxbrandi, there CI build checks are failing on the master branch too, it's independent of this pr. My pr (#32) was meant to fix them but it's not been merged...
I do not think it would make sense to sort a list of units. For instance how should ` [ 1m, 1kg ]` be sorted? My two cents say that...
Could this be solved by adding an `eval` function (or similar) to the debug object that would have this behavior. This would enable the desired behaviour without breaking code.
Possibly, although I don't know how well `eval` makes the purpose of the function clear. Also, users may confuse it with the built in eval function. `logResultOf(function)` is longer ,...
I guess the concern is that the function you pass to debug could have side effects. A forced example: ``` var x; debug.resultOf(() => (x = x || { run()...
@VBAssassin I made this for you. https://www.npmjs.com/package/@harrysarson/eval-debug
I put the warning in the readme to inform users of what could go wrong. Maybe it is worded a bit strongly-send a pr to me if you want to...