imporve pylint; report errors in a deterministic order
Bug #1625158 reported by
Amrith Kumar
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack DBaaS (Trove) |
Fix Released
|
Undecided
|
Peter Stachowski |
Bug Description
Peter (peterstac) observes that the fact that the config is not in a deterministic order makes it hard to compare two config's and see what changed.
Similarly, the output from pylint (errors) are generated one file at a time and os.walk makes no guarantee of deterministic order. So we should collect all errors (across all files) and then print an ordered list for human consumption.
Changed in trove: | |
assignee: | Amrith (amrith) → Peter Stachowski (peterstac) |
To post a comment you must log in.
Personally, I'm not positive I understand the first use-case; i.e. you have an existing config file, you save it, and then rebuild and then diff the two files. I'd have thought you'd just run check and the output of the tool was the diff.
I however do see the value in sorting the file so that when someone submits a change that includes a change to the config, reviewers can see more easily what the change is doing.