Join GitHub today
Sort text report output by file and line #1412
When Brakeman gives a lot of output, I find it can be really confusing to scroll through it and fix everything when its output is jumping between files seemingly at random. This is especially true where a new version of brakeman finds many new warnings, and the same warning occurs multiple times within the same file. Because Brakeman's output is not sorted by file/line, it appears as though Brakeman "missed" some warnings in the file until you scroll through its entire output.
This commit simply has Brakeman include filename and line number in its sorting so output containing lots of warnings appears more coherent.