Skip to content

Use danger-review from gitlab-org/quality/pipeline-common

Rémy Coutable requested to merge use-danger-review-from-template-3 into master

What does this MR do and why?

Third attempt at !121867 (merged), which was reverted by !121858 (merged) since it broke dev.

We now include the template with include:remote.

I verified that it works on dev: https://dev.gitlab.org/gitlab/gitlab-ee/-/merge_requests/1551/diffs

This depends on gitlab-org/quality/pipeline-common!308 (merged).

MR acceptance checklist

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

Edited by Rémy Coutable

Merge request reports