search

avaframe / AvaFrame

https://avaframe.org
European Union Public License 1.2
16 stars 2 forks source link

Adds a try except for cmcrameri lipari colormap (missing in older versions)[out3] #1035

Closed fso42 closed 3 days ago

fso42 commented 4 days ago

cmcrameri version 1.5 does not have lipari colormap, but is still widely used in oder QGis versions

pep8speaks commented 4 days ago

Hello @fso42! Thanks for opening this PR.

Line 979:121: E501 line too long (138 > 120 characters) Line 977:19: E712 comparison to False should be 'if cond is False:' or 'if not cond:' Line 948:21: E712 comparison to False should be 'if cond is False:' or 'if not cond:' Line 805:121: E501 line too long (124 > 120 characters) Line 689:37: E203 whitespace before ':' Line 688:37: E203 whitespace before ':' Line 537:15: E711 comparison to None should be 'if cond is not None:' Line 501:71: E203 whitespace before ':' Line 501:48: E203 whitespace before ':' Line 495:71: E203 whitespace before ':' Line 495:48: E203 whitespace before ':' Line 482:121: E501 line too long (121 > 120 characters)

Do see the Hitchhiker's guide to code style

codeclimate[bot] commented 4 days ago

Code Climate has analyzed commit 9c1bf7a8 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 50.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 69.6%.

View more on Code Climate.