ui replay: cleanup and fix workflow todos (#37230)
* fix: update pull request trigger and clean up workflow paths * fix other event names
This commit is contained in:
11
.github/workflows/ui_preview.yaml
vendored
11
.github/workflows/ui_preview.yaml
vendored
@@ -3,15 +3,14 @@ on:
|
||||
push:
|
||||
branches:
|
||||
- master
|
||||
pull_request: # TODO: change back to pull_request_target before merging
|
||||
pull_request_target:
|
||||
types: [assigned, opened, synchronize, reopened, edited]
|
||||
branches:
|
||||
- 'master'
|
||||
paths: # TODO: remove workflow path before merging
|
||||
paths:
|
||||
- 'selfdrive/assets/**'
|
||||
- 'selfdrive/ui/**'
|
||||
- 'system/ui/**'
|
||||
- '.github/workflows/ui_preview.yaml'
|
||||
workflow_dispatch:
|
||||
|
||||
env:
|
||||
@@ -101,7 +100,7 @@ jobs:
|
||||
uses: AnimMouse/setup-ffmpeg@ae28d57dabbb148eff63170b6bf7f2b60062cbae
|
||||
|
||||
- name: Finding diffs
|
||||
if: github.event_name == 'pull_request'
|
||||
if: github.event_name == 'pull_request_target'
|
||||
id: find_diff
|
||||
run: |
|
||||
export PYTHONPATH=${{ github.workspace }}
|
||||
@@ -139,7 +138,7 @@ jobs:
|
||||
} >> "$GITHUB_OUTPUT"
|
||||
|
||||
- name: Saving proposed ui
|
||||
if: github.event_name == 'pull_request'
|
||||
if: github.event_name == 'pull_request_target'
|
||||
working-directory: ${{ github.workspace }}/master_mici
|
||||
run: |
|
||||
git config user.name "GitHub Actions Bot"
|
||||
@@ -164,7 +163,7 @@ jobs:
|
||||
git push origin ${{ env.REPORT_FILES_BRANCH_NAME }}
|
||||
|
||||
- name: Comment on PR
|
||||
if: github.event_name == 'pull_request'
|
||||
if: github.event_name == 'pull_request_target'
|
||||
uses: thollander/actions-comment-pull-request@v2
|
||||
with:
|
||||
message: |
|
||||
|
||||
Reference in New Issue
Block a user