Skip to content

Commit

Permalink
Merge pull request statsmodels#9351 from bashtage/88-chars
Browse files Browse the repository at this point in the history
STY: Accept 88 characters in linting
  • Loading branch information
bashtage authored Sep 7, 2024
2 parents f8b2233 + 1f6b4ef commit 90b2b43
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
6 changes: 4 additions & 2 deletions lint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,9 @@ if [ "$LINT" == true ]; then
# pass _all_ flake8 checks
echo "Linting known clean files with strict rules"
# Default flake8 rules plus the additional rules from setup.cfg
flake8 --isolated --ignore=E121,E123,E126,E226,E24,E704,W503,W504,E741,E203 \
flake8 --isolated \
--max-line-length 88 \
--ignore=E121,E123,E126,E226,E24,E704,W503,W504,E741,E203 \
examples \
setup.py \
statsmodels/__init__.py \
Expand Down Expand Up @@ -377,7 +379,7 @@ if [ "$LINT" == true ]; then
if [ -n "$NEW_FILES" ]; then
echo "Linting newly added files with strict rules"
echo "New files: $NEW_FILES"
flake8 --isolated --ignore=E121,E123,E126,E226,E24,E704,W503,W504,E741,E203 $(eval echo $NEW_FILES)
flake8 --isolated --max-line-length 88 --ignore=E121,E123,E126,E226,E24,E704,W503,W504,E741,E203 $(eval echo $NEW_FILES)
if [ $? -ne "0" ]; then
echo "New files failed linting."
RET=1
Expand Down
3 changes: 2 additions & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,8 @@ markers =
junit_family = xunit2
[flake8]
exclude=.git,build,docs,archive
exclude = .git,build,docs,archive
max-line-length = 88
ignore=
W503,
# W503: line break before binary operator
Expand Down

0 comments on commit 90b2b43

Please sign in to comment.