Remove accreditation from check-cves and update finding logging logic
Background
In the VAT release we are planning to deploy this Thursday, accreditations will be removed and this update will prevents the check-cves from passing.
We need to remove the accreditation check to resolve this, but we should also clean up some of the code in this stage while we're working on it.
Acceptance Criteria
-
Remove accreditation check and approval check -
Create separate MR for this change (in case we can't get all of the other changes in)
-
-
Update findings logging to do the following -
Print unverified findings
-
-
Pull out the trigger for the approval badges -
Update check cves to generate badge
Definition of Done
-
Write or update any unit or integration tests -
Project pipeline runs successfully -
Add any applicable checkboxes for testing e.g. Ran pipeline in staging env or pipeline-test-project pipeline run -
Solution is captured as code and/or documentation and merge requests have been submitted -
Code review completed and merge request approved/merged -
All Acceptance Criteria have been completed
Edited by Kenneth Maguire