Sorting disfunctional with format csv and by-file option

[doomguy]

Describe the bug
Sorting (-s, --sort) doesn't work together with "-f csv" and "--by-file"

To Reproduce

$ scc --by-file -f csv --sort lines gitlab-pages | column -s, -t | head -n 4
Language    Location                                                                                                           Filename                        Lines  Code  Comments  Blanks  Complexity  Bytes
Makefile    gitlab-pages/Makefile.util.mk                                                                                      Makefile.util.mk                68     52    1         15      3           2447
Markdown    gitlab-pages/PROCESS.md                                                                                            PROCESS.md                      80     61    0         19      0           4575
License     gitlab-pages/LICENSE                                                                                               LICENSE                         19     16    0         3       0           1046

Expected behavior
The rows are sorted by lines.

Desktop (please complete the following information):

• OS: macOS
• Version 3.0.0
• Install Source: homebrew

Thanks for looking into it!
/Doomguy

[boyter]

Technically not dysfunctional. Its not present at all :P

I never put it in because, well CSV you should be able to sort it yourself :) I guess I can add it though.

[boyter]

@doomguy If you are able to build from source that should work for you as expected now.

[doomguy]

Can confirm - lines are now sorted. Thank you a lot 💜

I do however have the problem now that there is no header line anymore in CSVs:

# homebrew@3.0.0
% scc --by-file -f csv gitlab-pages | head -n1
Language,Location,Filename,Lines,Code,Comments,Blanks,Complexity,Bytes
# github master branch as of 2022-01-04
% ../../bin/scc/scc --by-file -f csv gitlab-pages | head -n1
Go,gitlab-pages/test/acceptance/zip_test.go,zip_test.go,377,332,7,38,12,11313

Was this intentional?

Regards
/Doomguy

[boyter]

100% something I missed. Its still in there, just sorted somewhere. Resolved now so headers should be both at the top. I did it correctly for the summary, just not the full files listing. A quick update should resolve this for you.

[doomguy]

Works now as desired. Thanks a lot! 💜