Skip to content

Commit

Permalink
ci: Add format check workflow
Browse files Browse the repository at this point in the history
Add a CI workflow to report formatting issues on changed files. This is
to gradually update files in the repository to be conform with PEP8
formatting.

Signed-off-by: Pieter De Gendt <[email protected]>
  • Loading branch information
pdgendt committed Nov 6, 2024
1 parent 5af0ecb commit 32b9e1b
Showing 1 changed file with 69 additions and 0 deletions.
69 changes: 69 additions & 0 deletions .github/workflows/format.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,69 @@
name: Format check

on:
pull_request:
branches:
- main
paths:
- '**.py'

jobs:
generate:
runs-on: ubuntu-latest
outputs:
files: ${{ steps.git-diff-filter.outputs.files }}
chunks: ${{ steps.git-diff-filter.outputs.chunks }}
name: Detect added and changed files

steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0

- uses: GrantBirki/git-diff-action@v2
id: git-diff-action
with:
base_branch: origin/main
search_path: '**.py'
git_options: '--no-color --full-index --unified=1'

- name: Filter json diff
id: git-diff-filter
env:
JSON_DIFF: ${{ steps.git-diff-action.outputs.json-diff }}
run: |
echo $JSON_DIFF | jq
files=$(echo $JSON_DIFF | jq -c -r '[.files[] | select(.type == "AddedFile") | {path: .path}]')
echo "files=$files" >> $GITHUB_OUTPUT
chunks=$(echo $JSON_DIFF | jq -c -r '[.files[] | select(.type == "ChangedFile") | . as $file | .chunks[] | select(.toFileRange.lines > 0) | {path: $file.path, start: .toFileRange.start, end: (.toFileRange.start + .toFileRange.lines)}]')
echo "chunks=$chunks" >> $GITHUB_OUTPUT
check-files:
needs: generate
if: ${{ needs.generate.outputs.files != '[]' }}
runs-on: ubuntu-latest
strategy:
matrix:
files: ${{ fromJSON(needs.generate.outputs.files) }}
name: Check file ${{ matrix.files.path }}
steps:
- uses: actions/checkout@v4
- uses: astral-sh/ruff-action@v1
with:
args: "format --check --diff"
src: "${{ matrix.files.path }}"

check-chunks:
needs: generate
if: ${{ needs.generate.outputs.chunks != '[]' }}
runs-on: ubuntu-latest
strategy:
matrix:
chunks: ${{ fromJSON(needs.generate.outputs.chunks) }}
name: "Check chunk ${{ matrix.chunks.path }}:${{ matrix.chunks.start }}-${{ matrix.chunks.end }}"
steps:
- uses: actions/checkout@v4
- uses: astral-sh/ruff-action@v1
with:
args: "format --check --diff --range ${{ matrix.chunks.start }}-${{ matrix.chunks.end }}"
src: "${{ matrix.chunks.path }}"

0 comments on commit 32b9e1b

Please sign in to comment.