Closed mfl15 closed 7 months ago
@mlupei do you mind providing me context on what this PR does? It looks like there was a lot of (manual? automatic?) reformatting that appears to not help readability in some places (though does help it on others), but it's inconsistently applied.
it is mostly automatic and some manual linter fixes, the purpose was to make the code more readable.
@mlupei do you mind providing me context on what this PR does? It looks like there was a lot of (manual? automatic?) reformatting that appears to not help readability in some places (though does help it on others), but it's inconsistently applied.
it is mostly automatic and some manual linter fixes, the purpose was to make the code more readable.
I agree that sometimes these changes are not really useful, but looks like overall it improves readability a bit.
Are there settings you can change in the linter? The consistent use of quotes is nice, and some of the formatting, but I ask since it is doing some strange things, eg. turning the simple
show_all = args.show_all # Show all organisms (no matter if present) in output file.
into the very odd line
show_all = (
args.show_all
) # Show all organisms (no matter if present) in output file.
Why the tuple surrounding the args.show_all
?
Are there settings you can change in the linter? The consistent use of quotes is nice, and some of the formatting, but I ask since it is doing some strange things, eg. turning the simple
show_all = args.show_all # Show all organisms (no matter if present) in output file.
into the very odd line
show_all = ( args.show_all ) # Show all organisms (no matter if present) in output file.
Why the tuple surrounding the
args.show_all
?
yes, sure, there is an ability to configure it, there is .pylintrc file.
Regarding to show_all case, yes, looks like it tried to divide the string and added the tuple, it should be reverted
Issues
3 New issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
Codecov Report
Attention:
103 lines
in your changes are missing coverage. Please review.Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #97 +/- ## ========================================== + Coverage 72.74% 77.42% +4.68% ========================================== Files 11 23 +12 Lines 1053 1608 +555 ========================================== + Hits 766 1245 +479 - Misses 287 363 +76 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.