Skip to content

Commit

Permalink
Ruff Formatting and add to CI (#5138)
Browse files Browse the repository at this point in the history
Co-authored-by: James Mayclin <[email protected]>
  • Loading branch information
johubertj and jmayclin authored Feb 26, 2025
1 parent a694bee commit ac1d098
Show file tree
Hide file tree
Showing 35 changed files with 2,633 additions and 1,398 deletions.
21 changes: 13 additions & 8 deletions .github/workflows/ci_linting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,20 +74,25 @@ jobs:
run: |
./codebuild/bin/run_kwstyle.sh
./codebuild/bin/cpp_style_comment_linter.sh
pepeight:
ruff:
runs-on: ubuntu-latest
steps:
- name: checkout
uses: actions/checkout@v4
- name: Run autopep8
id: autopep8
uses: peter-evans/autopep8@v2
with:
args: --diff --exit-code .

- name: Set up uv
uses: astral-sh/setup-uv@v5

- name: Run Ruff formatting check
working-directory: tests/integrationv2
id: ruff_format
run: uv run ruff format --check .
continue-on-error: true

- name: Check exit code
if: steps.autopep8.outputs.exit-code != 0
if: steps.ruff_format.outcome == 'failure'
run: |
echo "Run 'autopep8 --in-place .' to fix"
echo "Run 'ruff format .' to fix formatting issues"
exit 1
clang-format:
runs-on: ubuntu-latest
Expand Down
3 changes: 0 additions & 3 deletions .pep8

This file was deleted.

539 changes: 369 additions & 170 deletions tests/integrationv2/common.py

Large diffs are not rendered by default.

Loading

0 comments on commit ac1d098

Please sign in to comment.