Flake8 updating #1991

greglucas · 2022-01-22T20:45:33Z

Removing all of the flake8 ignores and updating the code appropriately.

All of the math still looks reasonable to me.

QuLogic

LGTM; some slight cleanup can be done:

lib/cartopy/io/srtm.py

lib/cartopy/tests/mpl/test_gridliner.py

lib/cartopy/tests/mpl/test_images.py

lib/cartopy/tests/test_util.py

greglucas · 2022-01-27T15:42:46Z

Thanks got those and added a few more from the test_utils.

greglucas force-pushed the flake8-updating branch from 1b9d9ae to 90e5b94 Compare January 22, 2022 20:55

greglucas added the Type: Infrastructure label Jan 25, 2022

greglucas added this to the 0.21 milestone Jan 25, 2022

greglucas added 2 commits January 25, 2022 08:26

MNT: Fixing flake8 E402 module imports

487a445

MNT: Fixing flake8 E241 multiple spaces after comma

9c3d92e

greglucas force-pushed the flake8-updating branch from 90e5b94 to fedc715 Compare January 25, 2022 15:26

QuLogic reviewed Jan 27, 2022

View reviewed changes

MNT: Fixing flake8 E225/E226 spaces after binary operators

91db588

greglucas force-pushed the flake8-updating branch from fedc715 to 91db588 Compare January 27, 2022 15:41

QuLogic approved these changes Jan 27, 2022

View reviewed changes

greglucas merged commit 22cdafc into SciTools:main Jan 29, 2022

greglucas deleted the flake8-updating branch January 29, 2022 00:37

Provide feedback