Skip to content

Commit 3a3b911

Browse files
authored
Merge pull request #314 from githubdoe/JST/fixReleaseWorflow
fix permissions for make-release.yml
2 parents 8dd673f + 6863a15 commit 3a3b911

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

.github/workflows/build-linux.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ jobs:
3838
- run: echo "::remove-matcher owner=uic-problem-matcher::"
3939

4040
- name: Run C++ Linter (clang-tidy)
41-
if: strategy.job-index == 0 #run only for first OS of the matrix
41+
if: strategy.job-index == 0 && github.event_name == 'pull_request' #run only for first OS of the matrix
4242
uses: cpp-linter/cpp-linter-action@v2
4343
env:
4444
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/make-release.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,9 @@ jobs:
3232
uses: ./.github/workflows/build-windows.yml
3333
# linux build is mainly here to check it builds. We have no acrtifact now.
3434
call-build-linux:
35+
permissions:
36+
contents: read
37+
pull-requests: write
3538
needs: check-semver
3639
uses: ./.github/workflows/build-linux.yml
3740

0 commit comments

Comments
 (0)