diff options
| author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2020-10-12 11:11:54 +0300 |
|---|---|---|
| committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2020-10-12 11:11:54 +0300 |
| commit | f8eccc7a0943143a0e414361f8b42089fb5dd193 (patch) | |
| tree | 54dff045ea8ce38d4c2ce1cbfce2030e763521e7 /danger | |
| parent | 99c7de6455d846a5fb98622b23afe151abc50bac (diff) | |
danger: Allow numbers in MR's title scope
An MRs title is allowed to have a scope like "praefect:" as prefix,
which helps narrowing down which area of the code is being changed. The
rule is quite strict though and doesn't allow for numbers in the scope,
but given our recent trips into Git2Go land this case does come up quite
often now.
Make the rule more lax and allow numbers as part of the scope.
Diffstat (limited to 'danger')
| -rw-r--r-- | danger/merge_request/Dangerfile | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/danger/merge_request/Dangerfile b/danger/merge_request/Dangerfile index 879d0f644..b9c81d2af 100644 --- a/danger/merge_request/Dangerfile +++ b/danger/merge_request/Dangerfile @@ -1,4 +1,4 @@ -unless /^([[:alpha:]][[:lower:]]+: )?[[:upper:]]/ =~ gitlab.mr_title +unless /^([[:alnum:]][[:lower:]]+: )?[[:upper:]]/ =~ gitlab.mr_title warn("Please capitalize the merge request title") end |