Skip to content

Commit 13a67aa

Browse files
Merge pull request #56 from DeveloperTryingToCodeLikeOtherOfThem/alert-autofix-8
Potential fix for code scanning alert no. 8: Workflow does not contain permissions
2 parents 37f03fc + 7f5a76a commit 13a67aa

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

.github/workflows/tag-bump-commit.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
# ref from pxt-arcade/.github/tag-bump-commit.yml
22

33
name: Tag version on merged bump commit
4+
permissions:
5+
contents: read
46

57
on:
68
workflow_call:
@@ -17,6 +19,7 @@ jobs:
1719
needs: check-merge
1820
runs-on: ubuntu-latest
1921
if: always()
22+
permissions: {}
2023
steps:
2124
- name: check-merge outputs
2225
run: |
@@ -27,6 +30,8 @@ jobs:
2730
needs: check-merge
2831
if: fromJSON(needs.check-merge.outputs.is_merged_pr || 'false') == true
2932
runs-on: ubuntu-latest
33+
permissions:
34+
contents: write
3035
outputs:
3136
did_tag: ${{ steps.tag-op.outputs.did_tag }}
3237
tag: ${{ steps.tag-op.outputs.tag }}
@@ -77,6 +82,7 @@ jobs:
7782
needs: check-merge
7883
if: fromJSON(needs.check-merge.outputs.is_merged_pr || 'false') == false
7984
runs-on: ubuntu-latest
85+
permissions: {}
8086
outputs:
8187
did_tag: false
8288
steps:
@@ -86,6 +92,7 @@ jobs:
8692
runs-on: ubuntu-latest
8793
needs: [tag-version, not-tag-version]
8894
if: always()
95+
permissions: {}
8996
outputs:
9097
did_tag: ${{ needs.tag-version.outputs.did_tag || false }}
9198
steps:

0 commit comments

Comments
 (0)