Skip to content

Commit a1f3928

Browse files
committed
Adjust CI config.
This is based on recommendations from zizmor, and followup reading on its audit checks.
1 parent 224f79f commit a1f3928

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

.github/workflows/ci.yml

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ name: CI
33

44
on:
55
push:
6-
branches: [trunk]
6+
branches: [ trunk ]
77
pull_request:
88
workflow_dispatch:
99

@@ -12,7 +12,7 @@ env:
1212
PIP_DISABLE_PIP_VERSION_CHECK: "1"
1313
PIP_NO_PYTHON_VERSION_WARNING: "1"
1414

15-
permissions: {}
15+
permissions: { }
1616

1717
jobs:
1818
build-package:
@@ -23,6 +23,7 @@ jobs:
2323
- uses: actions/checkout@v4
2424
with:
2525
fetch-depth: 0
26+
persist-credentials: false
2627

2728
- uses: hynek/build-and-inspect-python-package@v2
2829
id: baipp
@@ -59,7 +60,7 @@ jobs:
5960
- name: Run tests
6061
run: "python -Im nox --non-interactive --error-on-external-run --tag tests --python ${{ matrix.python-version }}"
6162
- name: Upload coverage data
62-
uses: actions/upload-artifact@v4
63+
uses: PaloAltoNetworks/upload-secure-artifact@v1.0.5
6364
with:
6465
name: coverage-data-${{ matrix.python-version }}
6566
path: .coverage.*
@@ -74,6 +75,8 @@ jobs:
7475

7576
steps:
7677
- uses: actions/checkout@v4
78+
with:
79+
persist-credentials: false
7780
- uses: actions/setup-python@v5
7881
with:
7982
python-version: "3.12"
@@ -96,7 +99,7 @@ jobs:
9699
coverage report --fail-under=100
97100
98101
- name: Upload HTML report if check failed.
99-
uses: actions/upload-artifact@v4
102+
uses: PaloAltoNetworks/upload-secure-artifact@v1.0.5
100103
with:
101104
name: html-report
102105
path: htmlcov

0 commit comments

Comments
 (0)