@@ -52,7 +52,7 @@ if [[ -z "$CHECK" || "$CHECK" == "lint" ]]; then
52
52
black --version
53
53
54
54
MSG=' Checking black formatting' ; echo $MSG
55
- black . --check
55
+ black . --check
56
56
RET=$(( $RET + $? )) ; echo $MSG " DONE"
57
57
58
58
# `setup.cfg` contains the list of error codes that are being ignored in flake8
@@ -104,7 +104,7 @@ if [[ -z "$CHECK" || "$CHECK" == "lint" ]]; then
104
104
isort --version-number
105
105
106
106
# Imports - Check formatting using isort see setup.cfg for settings
107
- MSG=' Check import format using isort ' ; echo $MSG
107
+ MSG=' Check import format using isort' ; echo $MSG
108
108
ISORT_CMD=" isort --recursive --check-only pandas asv_bench"
109
109
if [[ " $GITHUB_ACTIONS " == " true" ]]; then
110
110
eval $ISORT_CMD | awk ' {print "##[error]" $0}' ; RET=$(( $RET + ${PIPESTATUS[0]} ))
@@ -122,13 +122,18 @@ if [[ -z "$CHECK" || "$CHECK" == "patterns" ]]; then
122
122
# Check for imports from collections.abc instead of `from collections import abc`
123
123
MSG=' Check for non-standard imports' ; echo $MSG
124
124
invgrep -R --include=" *.py*" -E " from pandas.core.common import" pandas
125
+ RET=$(( $RET + $? )) ; echo $MSG " DONE"
125
126
invgrep -R --include=" *.py*" -E " from pandas.core import common" pandas
127
+ RET=$(( $RET + $? )) ; echo $MSG " DONE"
126
128
invgrep -R --include=" *.py*" -E " from collections.abc import" pandas
129
+ RET=$(( $RET + $? )) ; echo $MSG " DONE"
127
130
invgrep -R --include=" *.py*" -E " from numpy import nan" pandas
131
+ RET=$(( $RET + $? )) ; echo $MSG " DONE"
128
132
129
133
# Checks for test suite
130
134
# Check for imports from pandas.util.testing instead of `import pandas.util.testing as tm`
131
135
invgrep -R --include=" *.py*" -E " from pandas.util.testing import" pandas/tests
136
+ RET=$(( $RET + $? )) ; echo $MSG " DONE"
132
137
invgrep -R --include=" *.py*" -E " from pandas.util import testing as tm" pandas/tests
133
138
RET=$(( $RET + $? )) ; echo $MSG " DONE"
134
139
@@ -195,6 +200,10 @@ if [[ -z "$CHECK" || "$CHECK" == "patterns" ]]; then
195
200
invgrep -R --include=" *.py" --include=" *.pyx" -E ' class.*:\n\n( )+"""' .
196
201
RET=$(( $RET + $? )) ; echo $MSG " DONE"
197
202
203
+ MSG=' Check for use of {foo!r} instead of {repr(foo)}' ; echo $MSG
204
+ invgrep -R --include=* .{py,pyx} ' !r}' pandas
205
+ RET=$(( $RET + $? )) ; echo $MSG " DONE"
206
+
198
207
MSG=' Check for use of comment-based annotation syntax' ; echo $MSG
199
208
invgrep -R --include=" *.py" -P ' # type: (?!ignore)' pandas
200
209
RET=$(( $RET + $? )) ; echo $MSG " DONE"
@@ -203,6 +212,10 @@ if [[ -z "$CHECK" || "$CHECK" == "patterns" ]]; then
203
212
invgrep -R --include=* .{py,pyx} ' \.__class__' pandas
204
213
RET=$(( $RET + $? )) ; echo $MSG " DONE"
205
214
215
+ MSG=' Check for use of xrange instead of range' ; echo $MSG
216
+ invgrep -R --include=* .{py,pyx} ' xrange' pandas
217
+ RET=$(( $RET + $? )) ; echo $MSG " DONE"
218
+
206
219
MSG=' Check that no file in the repo contains trailing whitespaces' ; echo $MSG
207
220
INVGREP_APPEND=" <- trailing whitespaces found"
208
221
invgrep -RI --exclude=\* .{svg,c,cpp,html,js} --exclude-dir=env " \s$" *
0 commit comments