Skip to content

Commit

Permalink
Verify python source files with flake8.
Browse files Browse the repository at this point in the history
Fixes ceph#994

Signed-off-by: Gil Bregman <[email protected]>
  • Loading branch information
gbregman committed Dec 26, 2024
1 parent db5a1d9 commit 49741a4
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/build-container.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,12 @@ jobs:
with:
submodules: recursive

- name: Flake8 Linting
uses: lukacat10/Flake8@v5

- name: Verify Python source files
run: make verify

- name: Build container images - spdk
run: make build SVC="spdk" SPDK_TARGET_ARCH=x86-64-v2

Expand Down
6 changes: 6 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,12 @@ include mk/autohelp.mk
.DEFAULT_GOAL := all
all: setup $(ALL)

FLAKE_IGNORE = --extend-ignore E501,E251,E225,E711,E128,F541,W291,E211,E202,E265,E117,E111,E203,E127,E261,E222,E201,E262,E303,E302,E221,E231

verify: ## Run Python source files through flake8
@echo Verifying Python source files
flake8 $(FLAKE_IGNORE) control/*.py

setup: ## Configure huge-pages (requires sudo/root password)

@echo Setup core dump pattern as /tmp/coredump/core.*
Expand Down

0 comments on commit 49741a4

Please sign in to comment.