LGTM code review recommendations

prjemian commented 5 years ago

fixes #16

lgtm-com[bot] commented 5 years ago

This pull request fixes 1 alert when merging c0e0e92cd125690e1c60edde7036d25448a8394c into 9ad38900cadfa58dfbb0f5431798d3722c22d922 - view on LGTM.com

fixed alerts:

1 for Unused exception object

Comment posted by LGTM.com

lgtm-com[bot] commented 5 years ago

This pull request fixes 65 alerts when merging c5e3ae41f611ce60989e2e1b40c30a27190d1a07 into 9ad38900cadfa58dfbb0f5431798d3722c22d922 - view on LGTM.com

fixed alerts:

13 for Unused local variable
12 for Unused import
10 for Testing equality to None
9 for Variable defined multiple times
8 for Except block handles 'BaseException'
4 for Constant in conditional expression or statement
3 for Unreachable code
3 for Use of 'global' at module level
2 for Result of integer division may be truncated
1 for Unused exception object

Comment posted by LGTM.com

APS-USAXS / livedata

LGTM code review recommendations #17