Skip to content

Commit

Permalink
github: Move matrix-check.yml and patch-check.yml into lint.yml
Browse files Browse the repository at this point in the history
It does not really make sense to maintain these in separate workflows
when they run in the exact same fashion.

Signed-off-by: Nathan Chancellor <nathan@kernel.org>
  • Loading branch information
nathanchance committed Feb 14, 2022
1 parent caef323 commit a9e88e3
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 25 deletions.
25 changes: 22 additions & 3 deletions .github/workflows/lint.yml
@@ -1,9 +1,28 @@
# Run a couple of linters on code in repo
# * shellcheck and shfmt for shell scripts
# * YAPF for Python scripts
# Runs several checks
# * check-matrix.sh because GitHub limits a workflow matrix to 256 jobs
# https://docs.github.com/en/actions/reference/usage-limits-billing-and-administration#usage-limits
# * check-patches.sh to make sure applying patches via TuxSuite works correctly
# * shellcheck for shell script correctness
# * shfmt for shell script formatting consistency
# * yapf for Python formatting consistency

name: Lint checks
on: [push, pull_request]
jobs:
check_matrices:
name: Check workflow matrices
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: check-matrix.sh
run: bash check-matrix.sh
check_patch_series:
name: Check patches series
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: check-patches.sh
run: bash check-patches.sh
shellcheck:
uses: ClangBuiltLinux/actions-workflows/.github/workflows/shellcheck.yml@main
shfmt:
Expand Down
12 changes: 0 additions & 12 deletions .github/workflows/matrix-check.yml

This file was deleted.

10 changes: 0 additions & 10 deletions .github/workflows/patch-check.yml

This file was deleted.

0 comments on commit a9e88e3

Please sign in to comment.