Skip to content

Commit 12fd3d7

Browse files
authored
Merge pull request #370 from commerce-docs/ds_rm-reviewdog
Delete reviewdog from the PR testing workflow
2 parents 8b11c44 + 3ef879f commit 12fd3d7

File tree

3 files changed

+1
-29
lines changed

3 files changed

+1
-29
lines changed

.github/super-linter.env

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
FIX_MARKDOWN=true
21
IGNORE_GITIGNORED_FILES=true
32
MARKDOWN_CONFIG_FILE=.markdownlint.yml
43
VALIDATE_ALL_CODEBASE=false

.github/workflows/test-pull-request.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
permissions:
77
contents: read
88
statuses: write
9-
pull-requests: write
109

1110
jobs:
1211
testing:

.github/workflows/validate-pr_job.yml

Lines changed: 1 addition & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -13,41 +13,15 @@ jobs:
1313
with:
1414
fetch-depth: 0
1515

16-
- name: Setup reviewdog
17-
uses: reviewdog/[email protected]
18-
with:
19-
reviewdog_version: v0.20.3
20-
2116
- name: Load super-linter configuration
2217
shell: bash
2318
run: cat .github/super-linter.env >> "$GITHUB_ENV"
2419

25-
- name: Lint Code Base with super-linter and reviewdog
20+
- name: Lint Code Base with super-linter
2621
uses: super-linter/super-linter/[email protected]
2722
env:
2823
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
2924

30-
- name: Report linting results with reviewdog
31-
shell: bash
32-
run: |
33-
TMPFILE=$(mktemp)
34-
git diff > "${TMPFILE}"
35-
if [ ! -s "${TMPFILE}" ]; then
36-
echo "No changes detected, skipping reviewdog."
37-
exit 0
38-
fi
39-
git stash --include-untracked
40-
reviewdog \
41-
-f=diff \
42-
-f.diff.strip=1 \
43-
-name="markdownlint-fix" \
44-
-reporter="github-pr-review" \
45-
-filter-mode="diff_context" \
46-
-fail-level=error < "${TMPFILE}"
47-
git stash drop || true
48-
env:
49-
REVIEWDOG_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
50-
5125
remark-test:
5226
runs-on: ubuntu-latest
5327
needs: lint-test

0 commit comments

Comments
 (0)