dask / dask-image / 208

100%

main: 100%

push

travis-ci

Use flake8 (#93)

* Require flake8 be installed on CI

* Require `flake8` and `pytest-flake8` for testing

As the tests make use of flake8 (specifically a pytest plugin for
flake8), require `flake8` and `pytest-flake8`.

* Ensure flake8 is run with pytest

Uses the flake8 plugin for pytest to ensure that flake8 checks are run
as part of pytest.

* Exclude versioneer and docs from flake8

* Drop unused external imports caught by flake8

These imports were probably used for code that no longer needs them. So
go ahead and drop these unused imports.

* Wrap long warning line

Running flake8 picked out this line as too long. This wraps it some to
try and fix this issue.

* Break up warning string across multiple lines

To fit the 79 character limit constraint, split this warning into two
strings on two lines that are concatenated.

* Drop some internal unused imports

Some internal imports are unused according to flake8. So we drop them
here to fix these flake8 errors.

* Avoid flake8 unused import error

To workaround flake8 unused import errors where we know these functions
are imported and used elsewhere, go ahead and perform trivial
assignments to make use of these functions. This still passes the tests
and removes the flake8 errors.

* Reorder imports in ndfilters

Make sure that all imports in `ndfilters` happen before other statements
to fix flake8 errors.

* Ignore `lambda` assignment with flake8

We are not really interested in flake8's `lambda` assignment rule in
these cases. So simply tell flake8 to ignore these lines.

* Adjust line wrapping of `lambda`

Removes the need for the `\`, which seems to be causing issues with
`flake8`.

* Disable the flake8 lambda assignment on both lines

For some reason disabling this flake8 error on the second line is not
sufficient. So disable it on the first lines as well.

* Drop flake8 noqa from second `lambda` line

This `noqa` seems to have no e... (continued)

206 of 207 branches covered (99.52%)

Branch coverage included in aggregate %.

713 of 714 relevant lines covered (99.86%)

3.99 hits per line