Skip to content
This repository has been archived by the owner on Jul 28, 2023. It is now read-only.

Added support for the enableLoggingForApi property #127

Merged

Conversation

Blackbaud-BobbyEarl
Copy link
Contributor

Attempting to solve #126. This PR would depend on pghalliday/node-BrowserStackTunnel#31 passing, merged, and being released first.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

@Blackbaud-BobbyEarl
Copy link
Contributor Author

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

@Blackbaud-BobbyEarl
Copy link
Contributor Author

Blackbaud-BobbyEarl commented Apr 4, 2018

Support for the enableLoggingApi has been released in version 2.0.2 of node-BrowserStackTunnel. I've updated my PR to reflect that minimum version in package.json.

Is this PR something you could take a look at @dignifiedquire?

@dignifiedquire
Copy link
Member

thank you

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants