Skip to content

Commit

Permalink
Improve Linter
Browse files Browse the repository at this point in the history
create two separate linters, one for PRs, only checking
the added and changed files in the PR, and one for checking
the whole code base in the project.
  • Loading branch information
xx4h committed Apr 13, 2024
1 parent 0ef9f83 commit c71e5f0
Show file tree
Hide file tree
Showing 2 changed files with 36 additions and 7 deletions.
35 changes: 35 additions & 0 deletions .github/workflows/linter-full.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
name: Lint Pull Request

on:
workflow_dispatch:
pull_request:
branches: main

permissions: { }

jobs:
build:
name: Lint Pull Request
runs-on: ubuntu-latest
permissions:
contents: read
packages: read
# To report github actions status checks
statuses: write
steps:
- name: Checkout Code
uses: actions/checkout@v4
with:
# Full git history is needed to get a proper list of changed files within `super-linter`
fetch-depth: 0
- name: Lint Code Base
uses: super-linter/super-linter@v6
env:
# only validate new or edited files
VALIDATE_ALL_CODEBASE: false
VALIDATE_BASH_EXEC: false
VALIDATE_CPP: false
VALIDATE_CLANG_FORMAT: false
VALIDATE_SHELL_SHFMT: false
VALIDATE_PYTHON_PYLINT: false
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
8 changes: 1 addition & 7 deletions .github/workflows/linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,6 @@ on:
workflow_dispatch:
push:
branches: main
paths-ignore:
- '**/*.md'
pull_request:
branches: main
paths-ignore:
- '**/*.md'

permissions: { }

Expand All @@ -32,7 +26,7 @@ jobs:
uses: super-linter/[email protected]
env:
# only validate new or edited files
VALIDATE_ALL_CODEBASE: false
VALIDATE_ALL_CODEBASE: true
VALIDATE_BASH_EXEC: false
VALIDATE_CPP: false
VALIDATE_CLANG_FORMAT: false
Expand Down

0 comments on commit c71e5f0

Please sign in to comment.