Merge pull request #13616 from kaspersv/kaspersv/enable-implicit-this

Enable implicit this warning CI checks

Author: kaspersv

.github/workflows/check-implicit-this.yml

@@ -1,6 +1,13 @@
 name: "Check implicit this warnings"
 
-on: workflow_dispatch
+on:
+  workflow_dispatch:
+  pull_request:
+    paths:
+      - "**qlpack.yml"
+    branches:
+      - main
+      - "rc/*"
 
 jobs:
   check:
@@ -15,7 +22,7 @@ jobs:
           for pack_file in ${packs}; do
             option="$(yq '.warnOnImplicitThis' ${pack_file})"
             if [ "${option}" != "true" ]; then
-              echo "warnOnImplicitThis property must be set to 'true' for pack ${pack_file}"
+              echo "::error file=${pack_file}::warnOnImplicitThis property must be set to 'true' for pack ${pack_file}"
               EXIT_CODE=1
             fi
           done