Skip to content

Drop the spec in YamlProcessor that reads current .gitlab-ci.yml

Marin Jankovski requested to merge drop-ci-yaml-processor-test into master

What does this MR do?

Removes a spec.

Why was this MR needed?

In order to be able to use some licensed features, such as include in our .gitlab-ci.yml file, we need to supply a license. However, this spec only reads the current configuration file and checks if there are errors parsing it. If there were errors parsing the file however, CI wouldn't even run which makes this spec redundant. For this reason, it was discussed that we should at minimum remove the spec as it serves no purpose for the CI environment, and for the development environment it is better to set a fixture.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/48557

Merge request reports

Loading