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-...@eclipse.org>
  Reviewed-by: Mickael Istria <mist...@redhat.com>

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

_______________________________________________
platform-dev mailing list
platform-dev@eclipse.org
To unsubscribe from this list, visit 
https://www.eclipse.org/mailman/listinfo/platform-dev

Reply via email to