TWEAK

Author: marcoandries

.github/workflows/licensecheck.yml

@@ -53,20 +53,17 @@ jobs:
         run: echo "${{ steps.license-lines.outputs.license_lines }}"
 
       - name: Remove commit if licenses found
+        if: ${{ steps.license-lines.outputs.license_lines != '' }}
         id: remove-commit
         run: |
-            BAD_COMMIT=''
-            if ${{ steps.license-lines.outputs.license_lines != '' }}; then
-                BAD_COMMIT="${{ github.event.after }}"
-                echo "Removing commit $BAD_COMMIT as it contains licenses"
-                git reset --hard ${{ github.event.before }}
-                git push origin ${{ github.ref }} --force-with-lease
-            else
-                echo "NOT Removing commit"
-            fi
+            BAD_COMMIT="${{ github.event.after }}"
+            echo "Removing commit $BAD_COMMIT as it contains licenses"
+            git reset --hard ${{ github.event.before }}
+            git push origin ${{ github.ref }} --force-with-lease
             echo "bad_commit=$BAD_COMMIT" >> $GITHUB_OUTPUT
 
       - name: Find correspondences
+        if: ${{ steps.license-lines.outputs.license_lines != '' }}
         id: email
         uses: slackapi/[email protected]
         with:
@@ -90,13 +87,13 @@ jobs:
           token: ${{ secrets.SLACK_BOT_TOKEN }}
           payload: |
             "channel": "${{ env.SLACK_USER_ID }}",
-            "text": "${{ github.repository }} had a change!",
+            "text": "Commit ${{ steps.remove-commit.outputs.bad_commit }} was removed!",
             "blocks": [
               {
                 "type": "section",
                 "text": {
                   "type": "mrkdwn",
-                  "text": ":large_green_square: <https://github.com/${{ github.repository }}|*${{ github.repository }}*>: `${{ github.sha }}`"
+                  "text": ":alert-beam: <https://github.com/${{ github.repository }}/commits/${{ github.ref }}|*${{ github.repository }}>${{ github.ref }}*>: `${{ github.sha }}`"
                 }
               },
               {
@@ -107,4 +104,3 @@ jobs:
                 }
               }
             ]
-