Skip to content

Commit a5edaec

Browse files
ci: pass CODECOV_TOKEN via secrets inherit for PR builds
Switch conda-python-tests in pr.yaml to secrets: inherit so CODECOV_TOKEN reaches the shared-workflow codecov step. S3/AWS secrets are not needed for PR builds (nightly_report_helper skips S3 when RAPIDS_BUILD_TYPE!=nightly). Re-add run_codecov: false to test.yaml conda-python-tests since S3 explicit secrets are required there for nightly report aggregation, leaving no slot for CODECOV_TOKEN until rapidsai/shared-workflows#<TBD> lands. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Ramakrishna Prabhu <ramakrishnap@nvidia.com>
1 parent 3a9e2f3 commit a5edaec

2 files changed

Lines changed: 2 additions & 7 deletions

File tree

.github/workflows/pr.yaml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -416,13 +416,7 @@ jobs:
416416
with:
417417
build_type: pull-request
418418
script: ci/test_python.sh
419-
secrets:
420-
script-env-secret-1-key: CUOPT_S3_URI
421-
script-env-secret-1-value: ${{ secrets.CUOPT_S3_URI }}
422-
script-env-secret-2-key: CUOPT_AWS_ACCESS_KEY_ID
423-
script-env-secret-2-value: ${{ secrets.CUOPT_AWS_ACCESS_KEY_ID }}
424-
script-env-secret-3-key: CUOPT_AWS_SECRET_ACCESS_KEY
425-
script-env-secret-3-value: ${{ secrets.CUOPT_AWS_SECRET_ACCESS_KEY }}
419+
secrets: inherit # zizmor: ignore[secrets-inherit]
426420
docs-build:
427421
needs: [conda-python-build, changed-files]
428422
permissions:

.github/workflows/test.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ jobs:
5959
pull-requests: read
6060
uses: rapidsai/shared-workflows/.github/workflows/conda-python-tests.yaml@main
6161
with:
62+
run_codecov: false
6263
build_type: ${{ inputs.build_type }}
6364
branch: ${{ inputs.branch }}
6465
date: ${{ inputs.date }}

0 commit comments

Comments
 (0)