I think it should be changed, but I'm not sure how to change it. I think it should be changed because our git history is our legal trail of attribution. Mucking with it is relatively serious business.
As to how, normally I'd say git commit --amend --author "kylerichardson < kylerichards...@gmail.com>" if we act before the next commit and a git rebase otherwise, but it's pushed and rewriting history for a push'd commit has consequences. Not the least of which the scary force'd push. The challenge here is that all forked repos during this period between the wrong commit and the correction commit will be based on a dead branch. I guess I would vote for 1, the revert and then the re-commit. I'd like to understand a bit more about how this happened. Ryan, can you walk it through how you did the commit so we can avoid it in the future? Casey On Mon, Feb 27, 2017 at 4:04 PM, Kyle Richardson <kylerichards...@gmail.com> wrote: > Ok, so here's the story... Ryan was nice enough to commit my recent PR and > for whatever reason my github username but not my email address appears in > the commit author (see below). > > commit 41fc0ddc9881d9cfdd8bae129c0bb7800a116d4c > Author: kylerichardson <null> > Date: Mon Feb 27 11:38:55 2017 -0600 > > METRON-646 Add index templates to metron-docker (kylerichardson via > merrimanr) closes apache/incubator-metron#441 > > My question is can it be left as is or does it need to include the email > address per apache? > > If it needs to be changed, what are the acceptable options? > > (1) commit a revert and re-commit; maintains a record of everything > (2) rebase one back, update, and force a push; like it never happened > (3) another option I haven't considered? > > -Kyle >
