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.

Sign up for GitHub

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

Jump to bottom

boolean query param values are lowercased in the newest bravado library #19

Merged

sjaensch merged 1 commit into master from fix-bool-query-param-test

Aug 6, 2018

Owner

sjaensch commented Aug 6, 2018

We need to account for that.

It also looks like that aiohttp doesn't print request logs to stdout anymore, so I'm removing the redirection to /dev/null.


          boolean query param values are lowercased in the newest bravado libra…

c815a4f

…ry, account for that

sjaensch requested a review from macisamuele

August 6, 2018 12:10

coveralls commented Aug 6, 2018 •

edited

Coverage remained the same at 98.78% when pulling c815a4f on fix-bool-query-param-test into 2ccf5c4 on master.

macisamuele approved these changes

View reviewed changes

sjaensch merged commit 62dc29d into master

sjaensch deleted the fix-bool-query-param-test branch

August 6, 2018 16:14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment