Skip to content

Ensure we don't try to post an empty '/label' command

Rémy Coutable requested to merge fix-empty-label into master

What does this MR do and why?

Unlisted stage have an empty label set, so we tried to issue the /label quick action (since the label was empty). With this MR, we reject blank labels so that the quick action will actually be nil, and depending automations won't post anything.

Expected impact & dry-runs

These are strongly recommended to assist reviewers and reduce the time to merge your change.

See https://gitlab.com/gitlab-org/quality/triage-ops/-/tree/master/doc/scheduled#testing-policies-with-a-dry-run on how to perform dry-runs for new policies.

See https://gitlab.com/gitlab-org/quality/triage-ops/-/blob/master/doc/reactive/best_practices.md#use-the-sandbox-to-test-new-processors on how to make sure a new processor can be tested.

Action items

Edited by Rémy Coutable

Merge request reports