Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support complex GET query based the behavior of request module. #451

Merged
merged 4 commits into from
Jan 21, 2016

Conversation

BinChang
Copy link
Contributor

I found this issue #376, and feels it might be a good get-started-task to get my hands wet with open-source project, so I did this diff, please let me know if it would work.

@pgte @adborden

@pgte
Copy link
Member

pgte commented Jan 19, 2016

Tests are failing: https://travis-ci.org/pgte/nock/jobs/103245615

@BinChang
Copy link
Contributor Author

I forgot to update the package dependency. It should work now, Please take another look.

@pgte pgte merged commit 8f11c8d into nock:master Jan 21, 2016
@pgte
Copy link
Member

pgte commented Jan 21, 2016

@BinChang thanks!

Landed on v6.0.0.

@lock
Copy link

lock bot commented Sep 14, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue and add a reference to this one if it’s related. Thank you!

@lock lock bot locked as resolved and limited conversation to collaborators Sep 14, 2018
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.

None yet

2 participants