Lgtm alert fixes - Githubissues

BlueBrain / RTNeuron

Interactive visualization framework for geometrically detailed neuron simulations

GNU General Public License v3.0

30 stars 10 forks source link

Lgtm alert fixes #30

Closed hernando closed 5 years ago

hernando commented 5 years ago

This pull request fixes 14 alerts when merging af31e5b2b6a35efd00f7c978092c42dd5a1dd901 into 502b92ba7b500d4378d1198cef55b00ad5a24e9d - view on LGTM.com

fixed alerts:

4 for Unused local variable
4 for Unused import
3 for Testing equality to None
1 for Except block handles 'BaseException'
1 for 'import *' may pollute namespace
1 for Result of integer division may be truncated

Comment posted by LGTM.com

hernando commented 5 years ago

There are still 5 alerts that I thought I had fixed but they don't show as such.

hernando commented 5 years ago

This pull request fixes 14 alerts when merging 4c36724703a64a28f9beb10ed02b1daaf01f480e into 4323a1eb91b6263380cf3d202e936c5dd0925f7a - view on LGTM.com

fixed alerts:

4 for Unused local variable
4 for Unused import
3 for Testing equality to None
1 for Except block handles 'BaseException'
1 for 'import *' may pollute namespace
1 for Result of integer division may be truncated

Comment posted by LGTM.com