Update danger.js to pull_request_target (#2675)
This commit is contained in:
23
.github/workflows/ci-utils.yaml
vendored
23
.github/workflows/ci-utils.yaml
vendored
@ -3,7 +3,18 @@ on:
|
|||||||
push:
|
push:
|
||||||
branches:
|
branches:
|
||||||
- main
|
- main
|
||||||
pull_request:
|
# it's usually not recommended to use pull_request_target
|
||||||
|
# but we consider it's safe here if we keep the same steps
|
||||||
|
# see: https://securitylab.github.com/research/github-actions-preventing-pwn-requests/
|
||||||
|
# and: https://github.com/facebook/react-native/pull/34370/files
|
||||||
|
pull_request_target:
|
||||||
|
permissions:
|
||||||
|
actions: write
|
||||||
|
checks: write
|
||||||
|
contents: write
|
||||||
|
issues: write
|
||||||
|
pull-requests: write
|
||||||
|
statuses: write
|
||||||
jobs:
|
jobs:
|
||||||
danger-js:
|
danger-js:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
@ -19,13 +30,7 @@ jobs:
|
|||||||
node-version: "18"
|
node-version: "18"
|
||||||
- name: Utils / Install Dependencies
|
- name: Utils / Install Dependencies
|
||||||
run: cd packages/twenty-utils && yarn
|
run: cd packages/twenty-utils && yarn
|
||||||
- name: Set danger env
|
|
||||||
run: |
|
|
||||||
part1='github_pat_11BCSUEQA0Vd4ajqgGdujc_JJYRUxKfcbMtUn'
|
|
||||||
part2='z89DFTksGNXqOwLnmNv0VwMpQdc47T5BTGG2GOUbm6L8J'
|
|
||||||
echo "DANGER_GITHUB_API_TOKEN=${part1}${part2}" >> $GITHUB_ENV
|
|
||||||
- name: Utils / Run Danger.js
|
- name: Utils / Run Danger.js
|
||||||
run: cd packages/twenty-utils && yarn danger ci
|
run: cd packages/twenty-utils && yarn danger ci --use-github-checks --failOnErrors
|
||||||
env:
|
env:
|
||||||
DANGER_GITHUB_API_TOKEN: ${{ env.DANGER_GITHUB_API_TOKEN }}
|
DANGER_GITHUB_API_TOKEN: ${{ github.token }}
|
||||||
|
|
||||||
Reference in New Issue
Block a user