i'm happy to make any adjustments necessary, but this should be close. thanks for being open to the contribution

looks like i missed a simple syntax issue that npm t would have pointed out to me had i run it before pushing :\

ill have a fix up in a min

i'm having some trouble getting the tests to run correctly locally, but the travis output gives me proper insight with a longer feedback loop. i'll wrap those up tomorrow.

ah, it looks like a few tests that ensure the new config var is used somewhere in the code, but it doesnt seem to check files under bin/. do you want me to update the file search to include bin/ in addition to lib/?

Gonna cc @iarna on that q (I'm off work today)

sure thing. i'm happy to make the updates if thats best, but want to focus on what solution you see as most valuable.

enjoy the day off @zkat

i went ahead and included bin/ in the search for config usages and it does get the tests passing again. i made that change as a separate commit so i can easily back that out if you want to take a different approach. if you want to move forward with this approach, i can squash the two together, if you'd like.

anything else needed on my side to move this forward?

Everything looks good :)

thanks!