Skip to content

Commit

Permalink
Merge pull request CEED#1384 from CEED/jeremy/vermin
Browse files Browse the repository at this point in the history
Add Vermin in CI
  • Loading branch information
jeremylt authored Oct 20, 2023
2 parents 1d90b3a + e6aaec0 commit 3830ccc
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 3 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/python-test-with-style.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,3 +45,9 @@ jobs:
FC: gfortran-11
run: |
make format-py && git diff --exit-code
- name: Python version
env:
CC: ${{ matrix.compiler }}
FC: gfortran-11
run: |
make vermin
13 changes: 10 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -748,10 +748,10 @@ doc-html doc-latexpdf doc-epub doc-livehtml : doc-% : doxygen
doc : doc-html

# Style/Format
CLANG_FORMAT ?= clang-format
CLANG_FORMAT ?= clang-format
CLANG_FORMAT_OPTS += -style=file -i
AUTOPEP8 ?= autopep8
AUTOPEP8_OPTS += --in-place --aggressive --max-line-length 120
AUTOPEP8 ?= autopep8
AUTOPEP8_OPTS += --in-place --aggressive --max-line-length 120

format.ch := $(filter-out include/ceedf.h $(wildcard tests/t*-f.h), $(shell git ls-files '*.[ch]pp' '*.[ch]'))
format.py := $(filter-out tests/junit-xml/junit_xml/__init__.py, $(shell git ls-files '*.py'))
Expand All @@ -764,6 +764,13 @@ format-py :

format : format-c format-py

# Vermin - python version requirements
VERMIN ?= vermin
VERMIN_OPTS += -t=3.7- --violations

vermin :
$(VERMIN) $(VERMIN_OPTS) $(format.py)

# Tidy
CLANG_TIDY ?= clang-tidy

Expand Down
1 change: 1 addition & 0 deletions requirements-test.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
pytest
autopep8
vermin

0 comments on commit 3830ccc

Please sign in to comment.