Skip to content

Commit

Permalink
add pylintrc, update .tox fix files
Browse files Browse the repository at this point in the history
Signed-off-by: Yu Chin Fabian Lim <[email protected]>
  • Loading branch information
fabianlim committed May 15, 2024
1 parent a52a7af commit 630dd42
Show file tree
Hide file tree
Showing 10 changed files with 829 additions and 211 deletions.
16 changes: 10 additions & 6 deletions .github/workflows/format.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,9 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
framework_plugin:
plugin_name:
- "framework"
- "accelerated-peft"
# - "accelerated-peft" # enable later

steps:
- uses: actions/checkout@v4
Expand All @@ -39,7 +39,11 @@ jobs:
run: |
python -m pip install --upgrade pip
python -m pip install tox
- name: Change to plugin directory
run: cd plugins/${{ matrix.framework_plugin }}
- name: Run linter and formatter
run: tox -e lint
- name: Run linter
run: |
cd plugins/${{ matrix.plugin_name }}
tox -e lint
- name: Run formatter
run: |
cd plugins/${{ matrix.plugin_name }}
tox -e fmt
8 changes: 8 additions & 0 deletions plugins/accelerated-peft/tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,13 @@ commands =

[testenv:lint]
description = run linters
deps =
pylint>=2.16.2,<=3.1.0
commands = pylint src tests
allowlist_externals = pylint

[testenv:fmt]
description = format
skip_install = true
deps =
black>=22.12
Expand All @@ -26,6 +33,7 @@ commands =
black {posargs:.}
isort {posargs:.}


# [testenv:build]
# description = build wheel
# deps =
Expand Down
Loading

0 comments on commit 630dd42

Please sign in to comment.