Skip to content

Stop creating and testing combined.yml

Craig Smith requested to merge craigmsmith-fix-validity-check into main

What does this MR do?

The semgrep check is failing because the IDs in the combined.yml that the semgrep check tests are invalid. However, the combined.yml is an artifact of how the YAML was once created for export to semgrep. This method is no longer relevant, the yaml that is exported to semgrep is generated by the deploy script and stored in the dist directory. The semgrep check still tests the files in the dist directory so it's safe to remove the combined.yml.

This MR removes all references to the combined.yml and removes that part of the check.

What are the related issues?

gitlab-org/gitlab#426482 (closed)

Edited by Craig Smith

Merge request reports

Loading