Skip to content

Commit

Permalink
Remove cudf-pandas-integration and add notes on cancellation policy (#72
Browse files Browse the repository at this point in the history
)

This PR removes the cudf-pandas-integration special-casing because [that
repo is now
archived](https://github.com/rapidsai/cudf-pandas-integration/). Those
tests were migrated into cudf proper in
rapidsai/cudf#16645.

I have also added some notes on why we allow downstream build jobs to
proceed even if upstream ones fail based on
#11.
  • Loading branch information
vyasr authored Jan 27, 2025
1 parent f1f3cef commit e5d2383
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
4 changes: 0 additions & 4 deletions .github/workflows/get-run-info.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -68,10 +68,6 @@ jobs:
export BRANCH="${UCX_PY_BRANCH}"
fi
if [ "${JUST_REPO}" = "cudf-pandas-integration" ]; then
export BRANCH="main"
fi
SHA=$(gh api -q '.commit.sha' "repos/${REPO}/branches/${BRANCH}")
export SHA
Expand Down
7 changes: 7 additions & 0 deletions .github/workflows/nightly-pipeline.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,13 @@ concurrency:
group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.rapids_version }}
cancel-in-progress: true

# We intentionally allow build jobs to proceed even if their dependencies
# failed since the upstream failures may be unrelated. This is the reason for
# the many `if: ${{ !cancelled() }}` conditionals below. This short-circuits
# the `needs` key, see
# https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#jobsjob_idneeds.
# See the below links for more information on the decision
# https://github.com/rapidsai/workflows/pull/11
jobs:
get-run-info:
uses: ./.github/workflows/get-run-info.yaml
Expand Down

0 comments on commit e5d2383

Please sign in to comment.