From c0ccf09c057734ca1cf6ffe32826408217dae56c Mon Sep 17 00:00:00 2001 From: Bilka Date: Sat, 19 Aug 2023 02:50:58 +0200 Subject: [PATCH] AO3-6590 Add reviewdog for linting ERB files (#4604) * AO3-6590 Add reviewdog ERB lint runner * AO3-6590 Remove erb lint from Hound It doesn't work. And if that would be fixed in the future, it would be linting twice. * AO3-6590 Rename reviewdog workflow to be more generic --- .github/workflows/reviewdog.yml | 27 +++++++++++++++++++++++++++ .hound.yml | 4 ---- 2 files changed, 27 insertions(+), 4 deletions(-) create mode 100644 .github/workflows/reviewdog.yml diff --git a/.github/workflows/reviewdog.yml b/.github/workflows/reviewdog.yml new file mode 100644 index 00000000000..c86a098d193 --- /dev/null +++ b/.github/workflows/reviewdog.yml @@ -0,0 +1,27 @@ +# Based on https://github.com/tk0miya/action-erblint/blob/main/README.md#example-usage + +name: Reviewdog + +on: [pull_request] + +permissions: + checks: write + +jobs: + erb-lint: + name: ERB Lint runner + runs-on: ubuntu-latest + steps: + - name: Check out code + uses: actions/checkout@v3 + + - name: Set up Ruby and run bundle install + uses: ruby/setup-ruby@v1 + with: + bundler-cache: true + + - name: erb-lint + uses: tk0miya/action-erblint@667687e73b44e7b7a710a1204b180f49f80ebb5e + with: + use_bundler: true + reporter: github-pr-check # default diff --git a/.hound.yml b/.hound.yml index f544a38eee4..42909ccedb6 100644 --- a/.hound.yml +++ b/.hound.yml @@ -1,10 +1,6 @@ # Available linter versions: # http://help.houndci.com/en/articles/2461415-supported-linters -erblint: - enabled: true - config_file: .erb-lint.yml - jshint: config_file: .jshintrc ignore_file: .jshintignore