Skip to content

Commit 654e62c

Browse files
committed
Remove shellcheck
The github permissions are currently blocking PRs from forks from writing checks/review commands. We cannot use workflow permissions to give write permissions to runs from forks. Options are to shuffle the work into a workflow_run triggered after the lint, so we can use the workflow_permissions. Or use pull_request_target. Or use a PAT with write permissions. Signed-off-by: Robert Young <robertyoungnz@gmail.com>
1 parent dfe3011 commit 654e62c

File tree

1 file changed

+0
-14
lines changed

1 file changed

+0
-14
lines changed

.github/workflows/lint.yaml

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -54,20 +54,6 @@ jobs:
5454
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
5555
run: |
5656
mvn -B -Pci -T1.5C verify -DskipTests -Djapicmp.skip=true
57-
- name: shellcheck
58-
uses: reviewdog/action-shellcheck@4c07458293ac342d477251099501a718ae5ef86e
59-
with:
60-
github_token: ${{ secrets.GITHUB_TOKEN }}
61-
reporter: github-pr-review # Change reporter.
62-
path: "." # Optional.
63-
pattern: "*.sh" # Optional.
64-
level: "error"
65-
fail_level: "error"
66-
exclude: |
67-
"./.git/*"
68-
"**/target"
69-
check_all_files_with_shebangs: "false" # Optional.
70-
shellcheck_flags: "--external-sources --severity=error"
7157
- name: Save PR number to file
7258
if: github.event_name == 'pull_request'
7359
run: echo ${{ github.event.number }} > PR_NUMBER.txt

0 commit comments

Comments
 (0)