This was my first time reviewing the output of these scripts (which is *awesome*). However, I didn't fully understand the files and it took a little detective work and ultimately a call with Marco to really understand what I was looking at. To help others avoid this in the future I created a new README file that describes the files. Patch attached.

CommentFileSizeAuthor
#1 1086028-add-file-descriptions-1937826-1.patch2.41 KBjredding

Comments

jredding’s picture

jredding commented
StatusFileSize
new 1086028-add-file-descriptions-1937826-1.patch2.41 KB
marvil07’s picture

marvil07 commented
Status: Active » Fixed

First I added your patch, then I moved/get into a lot more details into the main README.txt file, so now it looks like this compiled.

Thanks!

jredding’s picture

jredding commented

Nice. That looks good!

What are your thoughts on adding the headers directly to the log/stat files? This way they'll be self describing. I can roll a patch but I don't know much about CodeSwarm and if a header would screw things up there (although we could always rip out line 1 before inputting into Codeswarm)

marvil07’s picture

marvil07 commented

It makes sense. I added an issue for it on #1939212: Add headers for CSV files.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.