Stephen Kelly wrote: > Simple, short and noiseless commits are easy to review, so please see if > you can follow suit there. In particular, I tried to review the commit "Rule database structure changes", but it seems to contain lots of unrelated things such as outputting 'discarded' instead of 'suppressed', and lots more. That is all noise and makes the commit hard to review. Please reduce that commit to the change in rules_SipTest.py plus whatever minimum change is needed to the python code to support that. The rest of the changes can either be follow-ups or can appear before the API change commit. Thanks, Steve.