Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

Fix CI #141

Merged
merged 2 commits into from
Aug 11, 2019
Merged

Fix CI #141

merged 2 commits into from
Aug 11, 2019

Conversation

fkorotkov
Copy link
Contributor

by fixing test data

@fkorotkov
Copy link
Contributor Author

There are 3 failures left that I cannot understand 😿

@fkorotkov
Copy link
Contributor Author

All failures are coming from test_fixes. Had another attempt to understand what's going on there but with no success. @stevepeak could you please take a look?

@blueyed
Copy link
Contributor

blueyed commented Aug 10, 2019

#180 has fixes for CI.

@drazisil drazisil mentioned this pull request Aug 10, 2019
@drazisil
Copy link
Contributor

There are 3 failures left that I cannot understand crying_cat_face

@fkorotkov

Looks like the test file did not account for whitespace. I'd already started fixing this before I was alerted to your PR, I'm sorry no one ever got back to you, I wanted to thank you for trying.

@drazisil drazisil merged commit b51710e into codecov:master Aug 11, 2019
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.

3 participants