Skip to content

Commit 2c8b35b

Browse files
authored
Fix PyOxidizer pinning (#231)
* Desperate flailing * yeah we need to persist credentials * !? * Add explanation * Get rid of vars again * Let's figure out the necessary permissions * Add contents write permission * No periods in names * Allow opening pull requests
1 parent 5e37440 commit 2c8b35b

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

.github/workflows/pin-for-pyoxidizer.yml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,15 @@
11
---
2-
name: Pin dependencies for oxidization.
2+
name: Pin dependencies for oxidization
33

44
on:
55
workflow_dispatch:
66
schedule:
77
# Run every 1st of Month at 3:25 UTC
88
- cron: "25 3 1 * *"
99

10+
permissions:
11+
contents: write
12+
pull-requests: write
1013

1114
jobs:
1215
pin:
@@ -25,7 +28,8 @@ jobs:
2528
steps:
2629
- uses: actions/checkout@v4
2730
with:
28-
persist-credentials: false
31+
# We need to persist credentials to possibly push the branch later.
32+
persist-credentials: true
2933
- uses: actions/setup-python@v5
3034
with:
3135
# Keep in-sync with ./pyoxidizer.yml
@@ -44,8 +48,6 @@ jobs:
4448
- name: Commit & push changes
4549
# Only push if changes exist
4650
if: steps.changes.outputs.count > 0
47-
env:
48-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
4951
run: |
5052
git config user.name github-actions
5153
git config user.email [email protected]

0 commit comments

Comments
 (0)