#12 processcp.py: handle "wrangler" being lowercase
Merged by bcotton. Opened by adamwill.
fedora-pgm/ adamwill/pgm_scripts lowercase-wrangler  into  main

Download 12.patch

The Change template uses "" as
a placeholder for various items, including the tracker bug link.
The script tries to handle when this placeholder is present and
not treat it like it's an actual tracker bug link (or issue
link or whatever).

In https://fedoraproject.org/wiki/Changes/SPDX_Licenses_Phase_3
for some reason a slightly different placeholder string -
"" - is used. That broke this
detection, because the detection looks for "Wrangler" with a
capital W. Let's handle this, in case anyone else does it.

Signed-off-by: Adam Williamson awilliam@redhat.com

Pull-Request has been merged by bcotton

Would you believe I thought the initial issue was being caused by a typo! So I added the angle brackets but never capitalised the W in Wrangler but also because I was in the right ballpark to diagnose the problem so Im taking this as a partial win ;-)

Thank you Adam for the PR & Ben for merging :)

hey, you were definitely close!

editing the page to capitalize the W would also have solved this indeed, but I kinda like the idea of making the script handle it just so it never happens again. I suspect what happened in this case is the page was copied forward from a much older proposal, when maybe the placeholder used lowercase, and that seems like it might happen again.

Metadata