RFR: 1495: /backport should be allowed in PRs which have been integrated [v2]
Guoxiong Li
gli at openjdk.org
Thu Jul 28 22:47:00 UTC 2022
On Thu, 28 Jul 2022 19:42:47 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
>> What about adding a parameter like `Pattern commitComment` to the method `findIntegratedCommitHash`?
>
> The drawback of that is that we then need to have a copy of the pattern everywhere we call the method. I think it's better to have something like a static method on PullRequest like `String commitHashMessage(Hash)`, which takes a hash and returns a String with the formatted message. IntegrateCommand can use that to format the relevant part of the comment.
Maybe someone has another more elegant way to solve this issue. But now I can't find such way, so it is good to adopt your idea now.
-------------
PR: https://git.openjdk.org/skara/pull/1341
More information about the skara-dev
mailing list