-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fix packaged version. #21
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( I do have some suggestions for making it better though... For recipe:
|
I wonder how it go to be like this... |
Seems like the fn vs. fn_suffix split goes back a ways. And presumably the
retro-bot doesn't handle it well.
…On Thu, Jan 3, 2019 at 11:32 AM Martin Durant ***@***.***> wrote:
I wonder how it go to be like this...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#21 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABQHIu4thtF0RDXZqmf4mC_HfCKhp2Tgks5u_j61gaJpZM4ZofJZ>
.
|
The circleci failures may be due to a dockerhub outage. |
Agreed. Better wait a little while. |
By the way, RTD build also failed on the last merge https://readthedocs.org/projects/fastparquet/builds/8294621/ |
Going to merge. I don't have a guess for the RTD build... A bit of googling suggests that can happen when you try to get the git hash for an empty repo, but that doesn't make sense here. |
Reran RTD, and it passed, so never mind. |
Fixup for #20
cc @martindurant.
We'll need to remember to reset the build number to 0 on the next build.