Skip to content

Commit bd85bc5

Browse files
authored
use 'rapids-init-pip' in wheel CI, other CI changes (#235)
## Description Proposes a batch of small, mostly-unrelated changes to building and packaging. These are all part of RAPIDS-wide efforts to make it easier to reproduce CI locally and to use artifacts from one project's CI in another project's CI. Contributes to rapidsai/shared-workflows#356 * explicitly provides an input for `script` to workflows using it, instead of relying on a default value coming from the workflow file Contributes to rapidsai/build-planning#179 * adds a call to `rapids-init-pip` near the beginning of all CI scripts that use `pip` Also updates `pre-commit` config to the latest `rapids-dependency-file-generator` version. It was a few versions behind, thought it'd be ok to bundle that update in with the rest of these changes.
1 parent a71f5f9 commit bd85bc5

File tree

5 files changed

+7
-1
lines changed

5 files changed

+7
-1
lines changed

.github/workflows/build.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ jobs:
3434
build_type: ${{ inputs.build_type || 'branch' }}
3535
branch: ${{ inputs.branch }}
3636
date: ${{ inputs.date }}
37+
script: ci/build_python.sh
3738
sha: ${{ inputs.sha }}
3839
# Package is pure Python and only ever requires one build.
3940
matrix_filter: 'map(select(.ARCH == "amd64" and (.LINUX_VER | test("centos")|not))) | sort_by(.PY_VER | split(".") | map(tonumber)) | [.[-1]]'

.github/workflows/pr.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ jobs:
3030
build_type: pull-request
3131
# This selects "ARCH=amd64 + the latest supported Python + CUDA".
3232
matrix_filter: map(select(.ARCH == "amd64")) | max_by([(.PY_VER|split(".")|map(tonumber)), (.CUDA_VER|split(".")|map(tonumber))]) | [.]
33+
script: ci/build_python.sh
3334
conda-python-tests:
3435
needs: conda-python-build
3536
secrets: inherit
@@ -38,6 +39,7 @@ jobs:
3839
build_type: pull-request
3940
# This selects "ARCH=amd64 + the latest supported Python + CUDA".
4041
matrix_filter: map(select(.ARCH == "amd64")) | max_by([(.PY_VER|split(".")|map(tonumber)), (.CUDA_VER|split(".")|map(tonumber))]) | [.]
42+
script: ci/test_python.sh
4143
wheel-build:
4244
needs: checks
4345
secrets: inherit

.pre-commit-config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ repos:
1515
args: ['--config=.flake8']
1616
files: jupyterlab_nvdashboard/.*$
1717
- repo: https://github.com/rapidsai/dependency-file-generator
18-
rev: v1.14.0
18+
rev: v1.18.1
1919
hooks:
2020
- id: rapids-dependency-file-generator
2121
args: ['--clean']

ci/build_wheel.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ package_name="jupyterlab-nvdashboard"
1010
# Configure sccache and set the date string
1111
source rapids-configure-sccache
1212
source rapids-date-string
13+
source rapids-init-pip
1314

1415
rapids-logger "Install Node.js required for building the extension front-end"
1516

ci/test_wheel.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33

44
set -eou pipefail
55

6+
source rapids-init-pip
7+
68
# Set the package name
79
package_name="jupyterlab-nvdashboard"
810

0 commit comments

Comments
 (0)