mainly PEP8 fixes - Githubissues

karllark / dust_attenuation

Astronomical Dust Attenuation

http://dust-attenuation.readthedocs.io/

BSD 3-Clause "New" or "Revised" License

4 stars 8 forks source link

mainly PEP8 fixes #42

Closed karllark closed 5 years ago

karllark commented 5 years ago

Moved the widget subdir to be in dust_attenuation/dust_attenuation so that it will be included in the packaging (only stuff in the dust_attenuation included).

Fixed a number of '#' comment lines that had an unprintable character after the '#'. Not sure where this came from, but fixed to make the code cleaner.

Removed some unused imports and variables identified by lgtm.

karllark commented 5 years ago

This pull request fixes 3 alerts when merging 33ab4ffc8e907e96f3c13b803db0bedbca78ed6f into 5735b49174819d9e247d00e631739fcec44e639e - view on LGTM.com

fixed alerts:

2 for Unused local variable
1 for Encoding error

Comment posted by LGTM.com

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 91

7 of 7 (100.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 99.662%

Totals
Change from base Build 89:	-0.002%
Covered Lines:	295
Relevant Lines:	296

💛 - Coveralls

karllark commented 5 years ago

@dcorre : not sure if you want to check this pull request. Nothing really important, just doing some cleanup. But maybe of interest. :-)

karllark commented 5 years ago

Going to merge. Keeping up-to-date. :-)