add error message for action

The error message will appear in the annotation of the github action.

Signed-off-by: Inho Oh <inho.oh@sk.com>

Author: webispy

README.md

@@ -6,34 +6,7 @@ The `checkpatch.pl` is a perl script to verify that your code conforms to the Li
 
 ## Action setup guide
 
-### Pull Request from owned repository
-
-.github/workflows/main.yml
-
-```yml
-name: checkpatch review
-on: [pull_request]
-jobs:
-  my_review:
-    name: checkpatch review
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v1
-    - name: Run checkpatch review
-      uses: webispy/checkpatch-action@master
-      env:
-        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-```
-
-The checkpatch action will posting comments for error/warning result to the PR conversation.
-
-![check](https://raw.githubusercontent.com/wiki/webispy/checkpatch-action/img/action_conversation_comment.png)
-
-The capture image above shows the comments on the lines of code and the comments on the commit message.
-
-### Pull Request from forked repository
-
-The Github action has a limitation that doesn't have write permission for PR from forked repository. So the action cannot write a comment to the PR.
+### Pull Request
 
 .github/workflows/main.yml
 

action.yml

@@ -1,5 +1,5 @@
 name: 'checkpatch.pl PR review'
-description: 'Codew review for PR using checkpatch.pl'
+description: 'Code review for PR using checkpatch.pl'
 branding:
   icon: 'check-square'
   color: 'purple'

review.sh

@@ -3,58 +3,14 @@
 # To debug the current script, please uncomment the following 'set -x' line
 #set -x
 
-# Argument
-COMMIT=$1
-
-# Get PR number
-PR=${GITHUB_REF#"refs/pull/"}
-PRNUM=${PR%"/merge"}
-
 # Generate email style commit message
 PATCHMAIL=$(git show --format=email $1 | checkpatch.pl --no-tree -)
 
-# Github REST API endpoints
-BODY_URL=https://api.github.com/repos/${GITHUB_REPOSITORY}/issues/${PRNUM}/comments
-CODE_URL=https://api.github.com/repos/${GITHUB_REPOSITORY}/pulls/${PRNUM}/comments
-
 # Internal state variables
 RESULT=0
 FOUND=0
 MESSAGE=
 
-# Write message to specific file and line
-function post_code_message()
-{
-    echo "POST to ${CODE_URL} with ${MESSAGE}"
-    curl ${CODE_URL} -s \
-        -H "Authorization: token ${GITHUB_TOKEN}" \
-        -H "Content-Type: application/json" \
-        -X POST --data "$(cat <<EOF
-{
-    "commit_id": "$COMMIT",
-    "path": "${FILE}",
-    "position": ${LINE},
-    "body": "${MESSAGE}"
-}
-EOF
-)"
-}
-
-# Write message to pull-request comment
-function post_comment_message()
-{
-    echo "POST to ${BODY_URL} with ${MESSAGE}"
-    curl ${BODY_URL} -s \
-        -H "Authorization: token ${GITHUB_TOKEN}" \
-        -H "Content-Type: application/json" \
-        -X POST --data "$(cat <<EOF
-{
-    "body": ":warning: ${COMMIT} - ${MESSAGE}"
-}
-EOF
-)"
-}
-
 #
 # checkpatch.pl result format
 # ---------------------------
@@ -107,17 +63,13 @@ do
         else
             # An empty line means the paragraph is over.
             if [[ -z $row ]]; then
-                if [[ ! -z "$GITHUB_TOKEN" ]]; then
-                    echo "Post comment to Github"
-                    if [[ -z $FILE ]]; then
-                        post_comment_message
-                    else
-                        post_code_message
-                    fi
+                if [[ -z $FILE ]]; then
+                    echo "::error ::${MESSAGE}"
                 else
-                    # Output empty line
-                    echo
+                    echo "::error file=${FILE},line=${LINE}::${MESSAGE}"
                 fi
+                # Output empty line
+                echo
 
                 # Code review found a problem.
                 RESULT=1