Skip to content

Commit d347eca

Browse files
authored
fix(ci): resolve all zizmor findings and add zizmor pre-commit checks (#680)
* feat(ci): add zizmor config and pre-commit * fix(ci): hash-pin all third-party actions * fix(ci): use explicit permissions per-job * fix(ci): fix artipacked * fix(ci): disable cache on uv install action
1 parent 9c97c77 commit d347eca

4 files changed

Lines changed: 26 additions & 6 deletions

File tree

.github/workflows/build-and-deploy.yml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,19 +22,22 @@ jobs:
2222
name: Build (and deploy)
2323
runs-on: ubuntu-latest
2424
steps:
25-
- uses: actions/checkout@v4
25+
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4.3.1
2626
with:
2727
fetch-depth: 0
28+
persist-credentials: false
2829

2930
- name: Install uv
30-
uses: astral-sh/setup-uv@v5
31+
uses: astral-sh/setup-uv@d4b2f3b6ecc6e67c4457f6d3e41ec42d3d0fcb86 # v5.4.2
32+
with:
33+
enable-cache: false
3134

3235
- name: Build
3336
env:
3437
DEPLOYMENT_DOCS_BUILD_STABLE: ${{ startsWith(github.event.ref, 'refs/tags/') && 'true' || 'false' }}
3538
run: uv run make dirhtml SPHINXOPTS="-W --keep-going -n"
3639

37-
- uses: aws-actions/configure-aws-credentials@v4
40+
- uses: aws-actions/configure-aws-credentials@7474bc4690e29a8392af63c5b98e7449536d5c3a # v4.3.1
3841
if: ${{ github.repository == 'rapidsai/deployment' && (github.event_name == 'push' || github.event_name == 'workflow_dispatch') }}
3942
with:
4043
role-to-assume: ${{ vars.AWS_ROLE_ARN }}

.github/workflows/pre-commit.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,17 @@ on:
44
push:
55
pull_request:
66

7+
permissions: {}
78
jobs:
89
checks:
910
name: "pre-commit hooks"
1011
runs-on: ubuntu-latest
1112
steps:
12-
- uses: actions/checkout@v4
13-
- uses: actions/setup-python@v5
13+
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4.3.1
14+
with:
15+
fetch-depth: 0
16+
persist-credentials: false
17+
- uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5.6.0
1418
with:
1519
python-version: "3.12"
16-
- uses: pre-commit/action@v3.0.1
20+
- uses: pre-commit/action@2c7b3805fd2a0fd8c1884dcaebf91fc102a13ecd # v3.0.1

.github/zizmor.yml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
rules:
2+
unpinned-uses:
3+
config:
4+
policies:
5+
# We require SHA-pinning for all workflows and actions _except_ for those from
6+
# rapidsai/shared-workflows and rapidsai/shared-actions
7+
"rapidsai/shared-workflows/*": any
8+
"rapidsai/shared-actions/*": any
9+
"*": hash-pin

.pre-commit-config.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,10 @@ repos:
4444
additional_dependencies: [tomli]
4545
exclude: "^.*.jsonlines$"
4646
args: ["--toml", "pyproject.toml", "--ignore-words-list=classfication"]
47+
- repo: https://github.com/zizmorcore/zizmor-pre-commit
48+
rev: v1.24.1
49+
hooks:
50+
- id: zizmor
4751

4852
default_language_version:
4953
python: python3

0 commit comments

Comments
 (0)