Fixing code style issues

APS-4ID-POLAR / polartools

Python package to read and process data from the APS Polar beamline.

https://polartools.readthedocs.io/en/latest/

Other

2 stars 2 forks source link

Fixing code style issues #12

Closed gfabbris closed 3 years ago

lgtm-com[bot] commented 3 years ago

This pull request fixes 1 alert when merging 172e638386583822b376d11f61bc1f4eb04c26a8 into 0021ca85cca2d4223b579b7da93e71f4286a76a5 - view on LGTM.com

fixed alerts:

1 for Suspicious unused loop iteration variable

lgtm-com[bot] commented 3 years ago

This pull request fixes 1 alert when merging 5299ae283e764659ba555ae48534966a2bcfba74 into 0021ca85cca2d4223b579b7da93e71f4286a76a5 - view on LGTM.com

fixed alerts:

1 for Suspicious unused loop iteration variable

lgtm-com[bot] commented 3 years ago

This pull request fixes 1 alert when merging fb9f3ea776960150e5928dc7c4643138fbbf4f7f into 0021ca85cca2d4223b579b7da93e71f4286a76a5 - view on LGTM.com

fixed alerts:

1 for Suspicious unused loop iteration variable

lgtm-com[bot] commented 3 years ago

This pull request fixes 1 alert when merging 2d3df05580543547d73cc21805a561f08d406ca5 into 0021ca85cca2d4223b579b7da93e71f4286a76a5 - view on LGTM.com

fixed alerts:

1 for Suspicious unused loop iteration variable

gfabbris commented 3 years ago

Note that I removed the pep257 from Codacy, it applies both D212 and D213 which are conflicting. Also, Codacy already uses pydocstyle, which should catch docstring issues.

We could turn it back on if we setup a .prospector.yml file, see: http://prospector.landscape.io/en/master/profiles.html