1803374 - Numbers in PR descriptions are considered bug numbers by the Bugzilla GitHub integration

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Extensions, defect)

Description

[Marco Castelluccio [:marco]]

For example, https://github.com/mozilla/pdf.js/pull/15765 was linked to https://bugzilla.mozilla.org/show_bug.cgi?id=15742 because its title is "[api-minor] Remove the TextLayer timeout parameter (PR 15742 follow-up)".

In pdf.js, we mention "bug XXX" in the description when we want to reference a bug.

Comment 1

[David Lawrence [:dkl]]

I am using the same regexp used in moz-phab and few other places so not sure why PR XXX would have triggered it but we may have to get very specific going forward and when the hook is triggered. So I can definitely change it to only trigger when "[Bb]ug XXX" is encountered and just ignore any other integer number.

Comment 2

[:glob ✱]

(In reply to David Lawrence [:dkl] from comment #1)

So I can definitely change it to only trigger when "[Bb]ug XXX" is encountered and just ignore any other integer number.

You'd want \b[Bb]ug[ -]\d+ as I've seen both Bug 1803374 and Bug-1803374 .

Comment 3

[BMO Automation]

Attachment: [mozilla-bteam/bmo] Bug 1803374 - Numbers in PR descriptions are considered bug numbers by the Bugzilla GitHub integration (#2010)

Comment 4

[David Lawrence [:dkl]]

Merged
https://github.com/mozilla-bteam/bmo/commit/e4a3044d439b87bcbd2d297a55541dbb0750fefe