Feature/reviewer improv 2

[tznind]

Proposed Changes

Summarise your changes, and include a link to the CHANGELOG diff.

Types of changes

What types of changes does your code introduce? Tick all that apply.

• Bugfix (non-breaking change which fixes an issue)
• New Feature (non-breaking change which adds functionality)
• Breaking Change (fix or feature that would cause existing functionality to not work as expected)
• Documentation-Only Update (if none of the other choices apply)
  • In this case, ensure that the message of the head commit from the source branch is prefixed with [skip ci]

Checklist

By opening this PR, I confirm that I have:

• Reviewed the contributing guidelines for this repository
• Ensured that the PR branch is in sync with the target branch (i.e. it is automatically merge-able)
• Updated any relevant API documentation
• Created or updated any tests if relevant
• Accurately updated the CHANGELOG
  • NOTE: This must include any changes to any of the following files: default.yaml, any of the RabbitMQ server configurations, GlobalOptions.cs
• Listed myself in the CONTRIBUTORS file 🚀
• Requested a review by one of the repository maintainers

Issues

If relevant, tag any issues that are expected to be resolved with this PR. E.g.:

• closes Report progress when loading very large error reports in reviewer #591
• closes 'Identical Rules' should not return matches where a the rule applies to a different column #593
• closes Group Outstanding Failures by column #598

[lgtm-com]

This pull request introduces 4 alerts when merging 4ae2d95 into cc7da4d - view on LGTM.com

new alerts:

• 2 for Possible loss of precision
• 2 for Missing Dispose call on local IDisposable

[tznind]

@howff please can you approve again, had merge conflicts in changelog and it has dismissed your review