Fix up some string concatenation, line length issues, etc. #645
Conversation
|
Hello @mulhod! Thanks for updating this PR.
Comment last updated at 2020-11-18 15:53:42 UTC |
Codecov Report
@@ Coverage Diff @@
## main #645 +/- ##
=======================================
Coverage 95.11% 95.11%
=======================================
Files 27 27
Lines 3093 3093
=======================================
Hits 2942 2942
Misses 151 151
Continue to review full report at Codecov.
|
b14a548 to
45d13af
Compare
|
Why the coverage check is failing I have no idea. When I click on the "Details" link, I don't see any change in coverage... Perhaps I'm not reading it correctly? |
desilinguist
left a comment
There was a problem hiding this comment.
Looks great! Thanks for taking this on @mulhod!
|
@bndgyawali If you get a chance, could you take a look at this? There are quite a few changes, but they are all cosmetic. |
Closes #636
Mainly, this PR has to do with cleaning up some multi-line string concatenations (or removing them altogether when possible). There are a couple other randomish improvements I made as well (like reducing the length of some very long lines or lines where it was easy to do, removing unused parameters from a function used by a test, etc.).