Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[platform-dev] Gerrit settings

I noticed today that Gerrit adds new footers into commit messages,
like:

Reviewed-on: https://git.eclipse.org/r/c/platform/eclipse.platform.text/+/181809
  Tested-by: Platform Bot <platform-bot@xxxxxxxxxxx>
  Reviewed-by: Mickael Istria <mistria@xxxxxxxxxx>

Can we please switch that off? This means that the commit submitted is
not my commit, even if my commit is based on origin/master and the
submit would be a simple fast-forward. After submit origin/master and my
local master will be on different commits. This is rather annoying if I
have additional commits (not yet pushed): I now have to rebase where
previously that was not necessary.

Besides, the usefulness of these footers is questionable:

Reviewed-on: the URL is not guaranteed to be stable. Gerrit has in the
past changed its path routes several times, which broke old links. The
commit already has a Change-Id: footer; with the Gerrit change-ID, the
review can be found always.

Tested-by: come on. We all know that our commits go through CI. Besides,
Gerrit has refs/notes for this.

Reviewed-by: Gerrit has refs/notes for that.

Cheers,

  Thomas



Back to the top