-
Notifications
You must be signed in to change notification settings - Fork 308
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: fix invalid version specification of pyarrow #1403
fix: fix invalid version specification of pyarrow #1403
Conversation
* ci: skip docfx in main 'Kokoro' presubmit * fix: specify default sessions in noxfile * add conditional for cd to project root
* ci: skip docfx in main 'Kokoro' presubmit * fix: specify default sessions in noxfile * fix: use python3 instead of 3.6 * fix: add NOX_SESSION to pass down envvars * fix: remove quotes arround sessions Co-authored-by: Tim Swast <[email protected]>
44e13e2
to
cc76951
Compare
🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use -- conventional-commit-lint bot |
cc76951
to
f104aff
Compare
f104aff
to
9d5f481
Compare
9d5f481
to
bb4d5f8
Compare
@aribray @chalmerlowe There seems to be something wrong with the build process. Could you tell me how to rerun kokoro in this PR and/or debug it on my local computer? I cherry-picked some commits from the main branch and they might solve the problem. Many thanks for considering my request. |
we have been doing some other changes in other PRs related to pyarrow, etc. |
Looking at the the current file on Am I missing something? Is this PR still relevant? |
@chalmerlowe Note that this is targeting the v1 branch. We have intentionally kept that branch alive for potential support for high priority fixes. Up to us if this qualifies. |
Make version specification of pyarrow fulfill PEP404.
Fixes #1402 🦕