[skara] RFR: 912: PR title correction logic can get stuck

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Thu Mar 11 12:02:31 UTC 2021

How come this email got a [skara] prefixed to the subject? But not the 
one for SKARA-913???

Hm... Checking my email history, there seems to be a handful of mails 
that has gotten a [skara] prefix, starting February 10th (but most of 
the mails do *not* get this). Did some configuration change happen that 
day? Is this prefix added by Skara mailing list bots, or by Mailman?

I find this kinds of prefix annoying, but the intermittency is certainly 
even more annoying!


On 2021-03-11 12:53, Magnus Ihse Bursie wrote:
> The PR title correct logic introduced with SKARA-572 can end up trying to correct the title forever. An observed case is where the JBS title ends with a trailing space. When trying to update the PR title to include this trailing space, it will be stripped by GitLab and no update will actually be done. This will then get retried again and again.
> The fix is to strip the JBS title before doing any comparisons.
> -------------
> Commit messages:
>   - 912: PR title correction logic can get stuck
> Changes: https://git.openjdk.java.net/skara/pull/1050/files
>   Webrev: https://webrevs.openjdk.java.net/?repo=skara&pr=1050&range=00
>    Issue: https://bugs.openjdk.java.net/browse/SKARA-912
>    Stats: 22 lines in 3 files changed: 17 ins; 0 del; 5 mod
>    Patch: https://git.openjdk.java.net/skara/pull/1050.diff
>    Fetch: git fetch https://git.openjdk.java.net/skara pull/1050/head:pull/1050
> PR: https://git.openjdk.java.net/skara/pull/1050

More information about the skara-dev mailing list