Skip to content

Commit

Permalink
ci(tools): Fix pushing python tools
Browse files Browse the repository at this point in the history
  • Loading branch information
lucasssvaz committed Jun 6, 2024
1 parent cc40edb commit 5fc499b
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions .github/workflows/build_py_tools.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,10 @@ jobs:
uses: actions/checkout@v4
with:
fetch-depth: 2
ref: ${{ github.event.pull_request.head.ref }}
- name: Verify Python Tools Changed
uses: tj-actions/changed-files@v41
id: verify-changed-files
with:
fetch_depth: '2'
since_last_remote_commit: 'true'
files: |
tools/get.py
Expand Down Expand Up @@ -72,6 +70,12 @@ jobs:
DISTPATH: pytools-${{ matrix.TARGET }}
PIP_EXTRA_INDEX_URL: "https://dl.espressif.com/pypi"
steps:
- id: create_token
uses: tibdex/github-app-token@v2
with:
app_id: ${{ secrets.TOOLS_UPLOAD_APP_ID }}
private_key: ${{ secrets.TOOLS_UPLOAD_APP_TOKEN }}

- name: List changed tools
shell: bash
run: |
Expand All @@ -89,8 +93,7 @@ jobs:
- name: Checkout repository
uses: actions/checkout@v4
with:
token: ${{ secrets.TOOLS_UPLOAD_PAT }}
ref: ${{ github.event.pull_request.head.ref }}
token: ${{ steps.create_token.outputs.token }}
- name: Set up Python 3.8
# Skip setting python on ARM because of missing compatibility: https://github.com/actions/setup-python/issues/108
if: matrix.os != 'ARM' && matrix.os != 'ARM64'
Expand Down

0 comments on commit 5fc499b

Please sign in to comment.