QuantEcon / QuantEcon.py / 28773098921

94%

push

github

CI: Fix flake8 --select arg so E231/F405 are actually enforced (#849)

The flake8 gate was written `flake8 --select F401, F405, E231 quantecon`.
The spaces after the commas make the shell split it into separate args, so
flake8 received `--select F401,` (trailing comma) plus `F405,`, `E231`,
`quantecon` as *positional paths*. Net effect: only F401 was selected, the
bogus `F405,`/`E231` paths were filtered out by the select, and the step
exited 0 -- E231 and F405 were never enforced in CI.

Fix the invocation to `flake8 --select=F401,F405,E231 quantecon`. Enabling
the gate surfaced 6 pre-existing E231 violations that had been slipping
through; fix those too so the corrected gate passes:

  - _compute_fp.py, util/numba.py: `{'cache':True}` -> `{'cache': True}`
  - _gridtools.py: `range(1,n+1)` -> `range(1, n+1)`
  - tests/test_gridtools.py: `{order:cartesian...}` -> `{order: cartesian...}`
  - markov/tests/test_approximation.py: space after `,` in the unpack
    continuation and in `map(np.array, (...))`

Verified: `flake8 --select=F401,F405,E231 quantecon` is clean, and the
affected test modules pass (39 passed). Only whitespace changed in the
source; no behavior change.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

1 of 1 new or added line in 1 file covered. (100.0%)

8093 of 8650 relevant lines covered (93.56%)

0.94 hits per line