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

Record errors branch #80

Open

kyrcha wants to merge 2 commits into master from record-errors-branch

Contributor

kyrcha commented Oct 25, 2019

Signed-off-by: Kyriakos Chatzidimitriou [email protected]

This PR addresses #13 issue. Adds:

tests retry for big queries producing 502
test errors that come with 200 OK status
refactors recordings

I have updated the CHANGELOG file according to the conventions in keepachangelog.com
This PR contains changes that do not require a mention in the CHANGELOG file

kyrcha added 2 commits

October 24, 2019 18:46


          Test retry, test some 200 errors, refactor recordings code

c82a1c6

Signed-off-by: Kyriakos Chatzidimitriou <[email protected]>


          Skip onilne retry tests on travis if token is unset

373485d

Signed-off-by: Kyriakos Chatzidimitriou <[email protected]>

kyrcha requested review from smacker and se7entyse7en

October 25, 2019 07:52

kyrcha self-assigned this

kyrcha mentioned this pull request

Test retry, test some 200 errors, refactor recordings code #78

Closed

2 tasks

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

Labels

None yet