GH-45155: [Python][CI] Fix path for scientific nightly windows wheel upload#45222
GH-45155: [Python][CI] Fix path for scientific nightly windows wheel upload#45222assignUser merged 7 commits intoapache:mainfrom
Conversation
|
|
|
@github-actions crossbow submit wheel-windows-cp313-cp313-amd64 |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
Revision: 423dce68311d05adb60fff48f51ffad1c78f5434 Submitted crossbow builds: ursacomputing/crossbow @ actions-7901efe159
|
assignUser
force-pushed
the
fix-sci-win
branch
from
423dce6 to
bf894ff
Compare
|
@github-actions crossbow submit wheel-windows-* This will upload the wheels, I'll remove the gate afterwards and merge so the nightly jobs work again. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
github-actions
bot
added
awaiting changes
Co-authored-by: Sutou Kouhei <[email protected]>
github-actions
bot
added
awaiting change review
|
@github-actions crossbow submit wheel-windows-* |
This comment was marked as outdated.
This comment was marked as outdated.
This reverts commit 46a2b0c.
|
Revision: a4047f4 Submitted crossbow builds: ursacomputing/crossbow @ actions-b7bf5250f6
|
This reverts commit bf894ff.
| echo "No wheel files found!" | ||
| exit 1 |
There was a problem hiding this comment.
Could you use 2-space indent like others?
| echo "No wheel files found!" | |
| exit 1 | |
| echo "No wheel files found!" | |
| exit 1 |
github-actions
bot
added
awaiting merge
raulcd
left a comment
raulcd
left a comment
There was a problem hiding this comment.
👍 I've learnt about compgen today
|
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 0d61cfd. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about 10 possible false positives for unstable benchmarks that are known to sometimes produce them. |
3 participants
Rationale for this change
The upload wasn't failing with the empty, non existent dir.
What changes are included in this PR?
Are these changes tested?
crossbow
Are there any user-facing changes?
no