Removes unnecessary checks from validator
What does this MR do and why?
Removes unnecessary checks from Validator
The merge request validator ensures security merge requests are ready to be processed. This commit removes two unnecessary validations:
- Milestone validation - This validation was added before the stable and version validation. When processing security merge requests, we cared about the version they are targeting not the milestone assigned. The milestone requirement is also not enforced in other checks
- Security label validation - Security merge requests that used the security template have the security label added by default. This label is not enforced in other processes so albeit informative it is not a requirement for security merge requests.
With these removals, we also aim to reduce the bureaucracy of security merge requests validations.
As a follow-up of gitlab-com/gl-infra/delivery#19856 (closed)
Author Check-list
- [-] Has documentation been updated?
Edited by Mayra Cabrera