18F / concourse-compliance-testing

Concourse CI assets for Compliance Toolkit
https://compliance-viewer.18f.gov/
Other
17 stars 7 forks source link

don't count any duplicate alerts in warnings #68

Closed afeld closed 8 years ago

afeld commented 8 years ago

In inspecting some of the C2 alerts, I was seeing that one result to another may vary by only a query string or something...not sure if we want to count that as two different alerts or not? Also, this change would only affect the Slack notification, not the results presented in Compliance Viewer. Therefore, not entirely sure if we want to merge this. Thoughts?

DavidEBest commented 8 years ago

👎 on anything that creates two sets of numbers. That'll just require that much more explaining.