Skip to content

Troubleshoot Travis #33280

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

Merged
merged 1 commit into from
Apr 4, 2020
Merged

Troubleshoot Travis #33280

merged 1 commit into from
Apr 4, 2020

Conversation

jbrockmendel
Copy link
Member

Reverts #33228, which appears to be the first build that started failing on Travis.

@jbrockmendel
Copy link
Member Author

@WillAyd looks like Travis passed (though it still shows yellow here, need to click through). any idea why the categorical test would affect the numexpr test?

@WillAyd
Copy link
Member

WillAyd commented Apr 4, 2020

On phone so haven’t checked in detail but seemingly unrelated?

@jbrockmendel
Copy link
Member Author

Any objection to merging this to get the CI fixed, then re-implementing #33228 in pieces to track down the issue?

cc @jreback

@jreback
Copy link
Contributor

jreback commented Apr 4, 2020

nope, do it when green

@BenjaminLiuPenrose
Copy link
Contributor

nope, do it when green

Hello @jreback any idea why CI test always fail for numexpr test?

@jbrockmendel
Copy link
Member Author

nope, do it when green

If you click through to Travis, it is green there (and has been for 16 hours), just hasnt de-yellowed here.

@jreback jreback merged commit cad602e into pandas-dev:master Apr 4, 2020
@jreback
Copy link
Contributor

jreback commented Apr 4, 2020

cool, seems we have seen this when travis status not updating before oh well

@jbrockmendel jbrockmendel deleted the travis-trouble branch April 4, 2020 17:17
jbrockmendel added a commit to jbrockmendel/pandas that referenced this pull request Apr 6, 2020
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.

4 participants