From 273013eab1768bf750bff069925dd44b48cc43ec Mon Sep 17 00:00:00 2001 From: lukaszgajewski Date: Tue, 20 Aug 2024 14:26:41 +0200 Subject: [PATCH] update lint action to follow black-flake8 interaction recommendation --- .github/workflows/lint_action.yml | 1 + README.md | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/lint_action.yml b/.github/workflows/lint_action.yml index 5169118..e761982 100644 --- a/.github/workflows/lint_action.yml +++ b/.github/workflows/lint_action.yml @@ -35,3 +35,4 @@ jobs: black_auto_fix: true flake8: true flake8_auto_fix: false + flake8_args: "--extend-ignore=E203 --max-line-length=88" diff --git a/README.md b/README.md index e57dba8..219fefd 100644 --- a/README.md +++ b/README.md @@ -31,7 +31,7 @@ def count_line(f, line): - Camel case with spaces for jupyter notebook: Analyze Brain Data.ipynb * Delete dead code! Don't outcomment code you don't use anymore, but delete it instead. If you need to find it again, that's what we have git for. * Use Jupyter Notebooks only for explanations and visualization. The actual programming should be happening in `.py` files. -* This repository is automatically set up with Github Actions/[Lint Action](https://github.com/marketplace/actions/lint-action) that will format your code using black and check for problems with flake8 ([without E203, W503](https://black.readthedocs.io/en/stable/faq.html#why-are-flake8-s-e203-and-w503-violated)). If either of them fails, you will not be able to merge your files unless you fix it. If this creates problem you cannot solve contact: florian@allfed.info +* This repository is automatically set up with Github Actions/[Lint Action](https://github.com/marketplace/actions/lint-action) that will format your code using black and check for problems with flake8 ([without E203](https://github.com/psf/black/blob/06ccb88bf2bd35a4dc5d591bb296b5b299d07323/docs/guides/using_black_with_other_tools.md#flake8)). If either of them fails, you will not be able to merge your files unless you fix it. If this creates problem you cannot solve contact: florian@allfed.info * You can deactivate the branch protection that makes sure that only correct and styled code can be merged, but it is not recommended.