-
Notifications
You must be signed in to change notification settings - Fork 170
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
v5 #111
Comments
Shorthands! 👍 |
mainly done. Gonna go for a release |
Ah no, blocked by node-fetch/node-fetch#132 |
Just voted it :) |
I just saw your commit about shorthands, what you think about |
If you want to add a PR for it after v5 is released I'll consider it if other users 👍 it. get, post, put, delete and head are the 'Big 5' methods, and as long as fetch-mock provides an api to target other methods I'm reasonably comfortable with leaving the shorthand coverage at that. I might add OPTINS too. What's your use case for PATCH? I've never had to use it |
Sure, I will add it :) |
greed
optioncatch
methodResponse
object as a response in.mock()
waiting on default to 200 response status node-fetch/node-fetch#132mockResponse()
internal as a utilityThe text was updated successfully, but these errors were encountered: