diff options
| author | Ian Cordasco <graffatcolmingov@gmail.com> | 2016-07-28 17:56:57 +0000 |
|---|---|---|
| committer | Ian Cordasco <graffatcolmingov@gmail.com> | 2016-07-28 17:56:57 +0000 |
| commit | ebc7ffd4e78e8f010c7ab12f2989fd614ef522d3 (patch) | |
| tree | b4cf3736ae8edba597dd7ca6e63a32b418b112a5 /src/flake8 | |
| parent | 53455fdff7e89f83d811feffcdd5238b84113d63 (diff) | |
| parent | ad3b4869095dc589c4c72982a101e6bdd6b5c87e (diff) | |
| download | flake8-ebc7ffd4e78e8f010c7ab12f2989fd614ef522d3.tar.gz | |
Merge branch 'fix-report-ordering' into 'master'
Sort reports by line and column
*Description of changes*
The reports have been incorrectly sorted. This fixes the sorting order to use the line number and then column. It also adds a test to verify that the sorting algorithm works.
*Related to:* #196
See merge request !100
Diffstat (limited to 'src/flake8')
| -rw-r--r-- | src/flake8/checker.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/flake8/checker.py b/src/flake8/checker.py index e7a94d8..d331b0c 100644 --- a/src/flake8/checker.py +++ b/src/flake8/checker.py @@ -306,7 +306,7 @@ class Manager(object): """ results_reported = results_found = 0 for checker in self.checkers: - results = sorted(checker.results, key=lambda tup: (tup[2], tup[3])) + results = sorted(checker.results, key=lambda tup: (tup[1], tup[2])) results_reported += self._handle_results(checker.display_name, results) results_found += len(results) |