[Tracker-discuss] [issue613] Commits notifications don't show up on bugs.p.o

[Ezio Melotti]

Ezio Melotti added the comment:

The commit message includes the pull request id, and it got accidentally captured by the regex that looks for issue numbers.  The code doesn't check if the issue id exists, and that caused the failure.

An easy fix is to add a try/except, but once we reach PR #1000 it will start matching and adding comments on old issues.  This might be good enough once we figure out a better solution.

Is the PR id always automatically added at the end of the commit message, between (...)?  If that's the case it might be easier to strip the end of the commit message, or only match for bpo-NNNNN once everyone starts using the convention.

_______________________________________________________
PSF Meta Tracker <metatracker at psf.upfronthosting.co.za>
<http://psf.upfronthosting.co.za/roundup/meta/issue613>
_______________________________________________________