ludeeus / action-shellcheck

GitHub action for ShellCheck.
MIT License
292 stars 69 forks source link

No annotations created for problems with severity "note" #46

Closed amezin closed 2 years ago

amezin commented 3 years ago

Describe the issue

In the default configuration, no annotations are created for "notes".

I guess it's because Github Actions don't understand what "note" is - https://github.com/actions/toolkit/blob/master/docs/problem-matchers.md:

severity: a group number containing either 'warning' or 'error' case-insensitive. Defaults to error

Links

amezin commented 3 years ago

https://docs.github.com/en/rest/reference/checks#annotations-items suggests that notice may be supported too (but not note that shellcheck outputs)

avindra commented 3 years ago

I switched to https://github.com/bewuethr/shellcheck-action and it works out of the box

amezin commented 3 years ago

I switched to https://github.com/bewuethr/shellcheck-action and it works out of the box

I haven't tried it (yet) but it looks like that action doesn't implement annotations at all

evulhotdog commented 2 years ago

I switched to bewuethr/shellcheck-action and it works out of the box

I haven't tried it (yet) but it looks like that action doesn't implement annotations at all

Just verified it does not.

evulhotdog commented 2 years ago

Reviewdog supports it, if you want shellcheck w/ Github Annotations.

  shellcheck:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v1
      - name: shellcheck
        uses: reviewdog/action-shellcheck@v1
        with:
          github_token: ${{ secrets.github_token }}
          reporter: github-pr-review
          path: "." # Optional.
          pattern: "*.sh" # Optional.
          filter_mode: nofilter # Optional.
          fail_on_error: true