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

feat(matcher): Allow bytes type in from_term function #281

Conversation

joshua-badger
Copy link
Contributor

Closes #241, wherein using bytes is not a valid option for either request or response content.

@joshua-badger joshua-badger force-pushed the add-bytes-support-for-from-term branch from 23ab5e9 to 1d8a225 Compare January 3, 2022 17:58
@elliottmurray
Copy link
Contributor

And this works for you? Just checking as I haven't run it locally

@joshua-badger
Copy link
Contributor Author

I am unable to run make test due to weirdness on OSX, but I can run the tests individually from Pycharm

@elliottmurray
Copy link
Contributor

More meant you were comfortable it is testing what you think it is. It looks like it.

@elliottmurray elliottmurray merged commit 891134a into pact-foundation:master Jan 3, 2022
@joshua-badger
Copy link
Contributor Author

Oh, yes. The traceback I ran into led me to that place, and we just need the return value to be what it is.

@joshua-badger
Copy link
Contributor Author

Thanks @elliottmurray !

@joshua-badger joshua-badger deleted the add-bytes-support-for-from-term branch January 3, 2022 18:25
@elliottmurray
Copy link
Contributor

No thank you!

At some point I'll fix my local env and trigger a release.

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

Successfully merging this pull request may close these issues.

Support bytes as response type
2 participants