Skip to content

Commit

Permalink
[ci] Move Verible lint action into CI workflow
Browse files Browse the repository at this point in the history
Signed-off-by: James Wainwright <james.wainwright@lowrisc.org>
  • Loading branch information
jwnrt committed Sep 30, 2024
1 parent 8d0e999 commit 5419cad
Show file tree
Hide file tree
Showing 2 changed files with 31 additions and 41 deletions.
31 changes: 31 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,3 +63,34 @@ jobs:
run: ./util/prep-bazel-airgapped-build.sh
- name: Build in the airgapped environment
run: ./ci/scripts/test-airgapped-build.sh

verible-lint:
name: Verible lint
runs-on: ubuntu-20.04
needs: quick_lint
env:
verible_config: hw/lint/tools/veriblelint/lowrisc-styleguide.rules.verible_lint
verible_version: v0.0-3430-g060bde0f
steps:
- uses: actions/checkout@v4
- name: Prepare Verible config
run: |
echo "Concatenating Verible waivers"
find . -type f -name '*.vbl' -exec cat {} \; >> verible_waiver
echo "::group::Verible config"
cat "$verible_config"
echo "::endgroup::"
echo "::group::Verible waiver"
cat "verible_waiver"
echo "::endgroup::"
- name: Run Verible linter action
uses: chipsalliance/verible-linter-action@v2.0
with:
github_token: ${{ secrets.GITHUB_TOKEN }}
verible_version: ${{ env.verible_version }}
reviewdog_reporter: 'github-pr-check'
suggest_fixes: 'false'
config_file: ${{ env.verible_config }}
extra_args: "--waiver_files=verible_waiver"
41 changes: 0 additions & 41 deletions .github/workflows/pr_lint.yml

This file was deleted.

0 comments on commit 5419cad

Please sign in to comment.