Skip to content

CI: pin xlrd <2 for now #38524

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

Closed
jreback opened this issue Dec 16, 2020 · 2 comments · Fixed by #38526
Closed

CI: pin xlrd <2 for now #38524

jreback opened this issue Dec 16, 2020 · 2 comments · Fixed by #38526
Labels
CI Continuous Integration Closing Candidate May be closeable, needs more eyeballs IO Excel read_excel, to_excel
Milestone

Comments

@jreback
Copy link
Contributor

jreback commented Dec 16, 2020

unless we merge the support indicated in #38522

I think we need to pin xlrd so we can have a green ci.

@jreback jreback added CI Continuous Integration IO Excel read_excel, to_excel labels Dec 16, 2020
@jreback jreback added this to the 1.2 milestone Dec 16, 2020
@jreback jreback mentioned this issue Dec 16, 2020
5 tasks
@jreback jreback reopened this Dec 16, 2020
@jreback
Copy link
Contributor Author

jreback commented Dec 16, 2020

actually let's leave this open until we revert

@jreback jreback modified the milestones: 1.2, 1.3 Dec 16, 2020
@fangchenli fangchenli added the Closing Candidate May be closeable, needs more eyeballs label May 4, 2021
@lithomas1 lithomas1 removed the Closing Candidate May be closeable, needs more eyeballs label May 4, 2021
@lithomas1 lithomas1 assigned lithomas1 and unassigned lithomas1 May 4, 2021
@lithomas1 lithomas1 added the Closing Candidate May be closeable, needs more eyeballs label May 4, 2021
@lithomas1
Copy link
Member

Fixed by #38571

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration Closing Candidate May be closeable, needs more eyeballs IO Excel read_excel, to_excel
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants