3 novembre 2020 00:10 "Ryosuke Niwa" <rn...@webkit.org> a écrit:
> On Mon, Nov 2, 2020 at 2:04 PM Jonathan Bedard <jbed...@apple.com> wrote: > >> We appreciate everyone’s feedback on transitioning away from Subversion to >> Git, I’ll be releasing >> an expected timeline of up-coming changes in the next week before the >> contributors meeting. >> >> In the mean time, we’re preparing on adding identifiers to new commit >> messages, that work is >> tracked in https://bugs.webkit.org/show_bug.cgi?id=218407. At the moment, >> we’re likely going to be >> appending the identifiers to commit messages (as the current change >> proposes), but I wanted to >> provide a chance for folks to object to this change before it becomes >> canonical. > > I'm a bit confused here. It looks like the patch only affects commits > made via webkit-patch. Given there are a lot of people who don't use > webkit-patch land, I'm not certain this strategy is sound even in the > short term. Why don't we do this in post commit hook instead? Yes, if your reason for switching to Git is making things easy for people using Git to contribute, please try to use the standard Git commands as much as possible. Otherwise, from a developer experience point of view you are still doing your custom system, and it does not matter that the backend is Git. You can use an approach similar to the hook used for Gerrit: https://gerrit-review.googlesource.com/Documentation/cmd-hook-commit-msg.html Note that this hook runs locally on developer's machine, but you can additionally have a server-side hook that checks that the id is indeed present in the commit message, and rejects the commit otherwise (Gerrit also does this for its Change-Id). I think it is not possible to alter the commit message from server-side with a standard Git server, it would create confusion because the local branch from the developer machine would not match what's on the server. -- Adrien. _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-dev