Skip to content

Conversation

@alimcmaster1
Copy link
Member

@alimcmaster1 alimcmaster1 commented Jan 5, 2020

hmm.. im not sure on this. But don't see any usage of this ret value.

The C extensions now build with this change

Potentially related change on the numpy side:
numpy/numpy#15232

cc. @WillAyd @jreback

@alimcmaster1 alimcmaster1 added CI Continuous Integration Dependencies Required and optional dependencies labels Jan 5, 2020
@alimcmaster1 alimcmaster1 changed the title CI: Fix numpydev build attempt... CI: Fix numpydev build Jan 5, 2020
@jbrockmendel
Copy link
Member

LGTM, thanks for tracking this down

@jreback jreback added this to the 1.0 milestone Jan 5, 2020
@jreback jreback merged commit 6f96331 into pandas-dev:master Jan 5, 2020
@jreback
Copy link
Contributor

jreback commented Jan 5, 2020

this is fine

that return value is from a really old version

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

Labels

CI Continuous Integration Dependencies Required and optional dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: Numpy Dev Build Failing

3 participants