Skip to content

Commit 78e7aca

Browse files
timkpaineCarreau
authored andcommitted
Merge pull request #211 from Point72/tkp/checklint
Check for python formatting changes in lint step
2 parents 8a0d881 + 086c9d5 commit 78e7aca

File tree

3 files changed

+16
-3
lines changed

3 files changed

+16
-3
lines changed

.github/workflows/build.yml

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -625,6 +625,20 @@ jobs:
625625
run: make test
626626
if: ${{ contains( 'numpy', matrix.package )}}
627627

628+
###########################
629+
#~~~~~~~~~~~~~~~~~~~~~~~~~#
630+
#~~~~~~|#############|~~~~#
631+
#~~~~~~|#|~~~~~~~/##/~~~~~#
632+
#~~~~~~|#|~~~~~/##/~~~~~~~#
633+
#~~~~~~~~~~~~/##/~~~~~~~~~#
634+
#~~~~~~~~~~/##/~~~~~~~~~~~#
635+
#~~~~~~~~/##/~~~~~~~~~~~~~#
636+
#~~~~~~/##/~~~~~~~~~~~~~~~#
637+
#~~~~~~~~~~~~~~~~~~~~~~~~~#
638+
# Test Service Adapters #
639+
#~~~~~~~~~~~~~~~~~~~~~~~~~#
640+
# Coming soon!
641+
628642
#############################
629643
#~~~~~~~~~~~~~~~~~~~~~~~~~~~#
630644
#~~~~~~|#############|~~~~~~#

Makefile

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ install: ## install library
4141
lint-py:
4242
python -m isort --check csp/ examples/ setup.py
4343
python -m ruff check csp/ examples/ setup.py
44+
python -m ruff format --check csp/ examples/ setup.py
4445

4546
lint-cpp:
4647
# clang-format --dry-run -Werror -i -style=file `find ./cpp/ -name "*.*pp"`
@@ -194,9 +195,6 @@ dependencies-debian: ## install dependencies for linux
194195
dependencies-fedora: ## install dependencies for linux
195196
yum install -y automake bison ccache cmake curl flex perl-IPC-Cmd tar unzip zip
196197

197-
dependencies-alma: ## install dependencies for linux
198-
dnf install -y automake bison ccache cmake curl flex perl-IPC-Cmd tar unzip zip
199-
200198
dependencies-vcpkg: ## install dependnecies via vcpkg
201199
cd vcpkg && ./bootstrap-vcpkg.sh && ./vcpkg install
202200

docs/wiki/dev-guides/Build-CSP-from-Source.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -227,6 +227,7 @@ make lint-py
227227
# or
228228
# python -m isort --check csp/ setup.py
229229
# python -m ruff check csp/ setup.py
230+
# python -m ruff format --check csp/ setup.py
230231
```
231232

232233
**Python Autoformatting**

0 commit comments

Comments
 (0)