search

pghalliday / node-BrowserStackTunnel

https://www.npmjs.com/package/browserstacktunnel-wrapper
MIT License
17 stars 24 forks source link

Added support for the enable-logging-for-api flag #31

Closed Blackbaud-BobbyEarl closed 6 years ago

Blackbaud-BobbyEarl commented 6 years ago

Attempting to solve #30

Blackbaud-BobbyEarl commented 6 years ago

Adding the branching logic to pass in --enable-logging-for-api caused the maxcomplexity jshint rule to fail. As it currently stands, I've increased the setting by 1. If you prefer, I can refactor my PR to accommodate the current setting of 15. If that is your preference, I would appreciate some guidance on refactoring just my contribution vs all the param checks.

pghalliday commented 6 years ago

thanks, will release a new version shortly