Skip to content

Commit d2254e4

Browse files
Thomas-Boiamacado
andauthored
Fix svg checker double posting comment (#660)
Co-authored-by: Clemens Bastian <8781699+amacado@users.noreply.github.com>
1 parent 63ca95a commit d2254e4

1 file changed

Lines changed: 0 additions & 21 deletions

File tree

.github/workflows/post_check_svgs_comment.yml

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -35,22 +35,6 @@ jobs:
3535
with:
3636
path: ./svg_err_messages/svg_err_messages.txt
3737

38-
# - name: Comment on the PR about the result - Success
39-
# uses: jungwinter/comment@v1 # let us comment on a specific PR
40-
# if: success() && steps.err_message_reader.outputs.content == '1'
41-
# env:
42-
# MESSAGE: |
43-
# Hi!
44-
# I'm Devicons' SVG-Checker Bot and everything looks great. Good job!
45-
46-
# Have a nice day,
47-
# SVG-Checker Bot :grin:
48-
# with:
49-
# type: create
50-
# issue_number: ${{ steps.pr_num_reader.outputs.content }}
51-
# token: ${{ secrets.GITHUB_TOKEN }}
52-
# body: ${{ env.MESSAGE }}
53-
5438
- name: Comment on the PR about the result - SVG Error
5539
uses: jungwinter/comment@v1 # let us comment on a specific PR
5640
if: success() && (steps.err_message_reader.outputs.content != '0' && steps.err_message_reader.outputs.content != '1')
@@ -79,11 +63,6 @@ jobs:
7963
token: ${{ secrets.GITHUB_TOKEN }}
8064
body: ${{ format(env.MESSAGE, steps.err_message_reader.outputs.content) }}
8165

82-
# if we posted error messages before, we fail this entire workflow
83-
- name: Fail workflow is there is an error.
84-
uses: cutenode/action-always-fail@v1.0.1
85-
if: success() && (steps.err_message_reader.outputs.content != '0' && steps.err_message_reader.outputs.content != '1')
86-
8766
- name: Comment on the PR about the result - Failure
8867
uses: jungwinter/comment@v1 # let us comment on a specific PR
8968
if: failure()

0 commit comments

Comments
 (0)