danger: Don't warn for missing labels while adding some
requested to merge 343748-danger-can-prompt-for-type-label-even-when-danger-applies-the-type-label-itself into master
What does this MR do and why?
This changes the Danger rules that adds labels to only add the labels to an array, and let the main Dangerfile handle the actual API request to add the labels (with a single gitlab.api.update_merge_request instead of potentially 4 different ones before).
The main reason for this change is to prevent Danger from warning about a missing MR type label, while the label is actually added in the same job.
Screenshots or screen recordings
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #343748 (closed).
Edited by Rémy Coutable