Skip to content

Move VulnerabilityFindingDismiss to correct file location

Jonathan Schafer requested to merge 367402-move-dismiss-security-finding into master

What does this MR do and why?

This MR is the first part of completing #367402 (closed). I split up the issue into to parts as such:

  • Moving the VulnerabilityFindingDismiss to the vulnerabilities/finding directory, more appropriate with its behavior of dismissing vulnerability findings. (this MR)
  • Creation of a new mutation, DismissPipelineSecurityFinding, to dismiss a security finding from the CI pipeline, which requires this MR to be merged first. (Create GraphQL mutation to dismiss Security Fin... (!93302 - merged))

VulnerabilityFindingDismiss is being deprecated, but I needed to get it in the right place so I could properly do the new mutation. I chose to split them up for reviewing simplicity. This MR only moves files, there is no new code.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

  1. Run VulnerabilityFindingDismiss as currently implemented, there should be no change in behavior.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #367402 (closed)

Merge request reports