gitmask icon indicating copy to clipboard operation
gitmask copied to clipboard

Option to preserve commit logs

Open ikedam opened this issue 7 years ago • 5 comments
trafficstars

gitmask squashes all commits into one commit with the message "anonymous commit" by ghost. Though it removes identifications completely, it also loses much information in the change. Commit messages are often important in development.

It would be nice if there's an option to preserve commit logs and replaces the committer and the author with ghost to remove identifications:

  • Maintainers can get the outline of the request from commit messages.
  • The pull request can be more descriptive by splitting commits in appropriate logical units.

I believe it can be done by running this command in the branch created from the bundle instead of merging that branch into the target branch and squashing commits:

git rebase -f  ${destBranchName} -x "git commit --amend --reset-author"

ikedam avatar Jul 15 '18 10:07 ikedam

This is definitely an interesting option, however I think the default should still be squashing the commits together. It's possible to use commit datetime analysis to narrow down probable author countries.

We could add options via query string parameters, but I'm not sure if they would be associated as s3 file metadata.

AnalogJ avatar Feb 23 '19 17:02 AnalogJ

the default should still be squashing the commits together.

I agree. This should be provided with an option parameter. I’ll see I can use s3 meta data, like tags.

ikedam avatar Feb 24 '19 21:02 ikedam

I can see this being useful as well. @ikedam Are you working on this?

j-rewerts avatar Apr 05 '19 18:04 j-rewerts

@j-rewerts Unfortunately, not yet. I’m a newbie to nodejs and serverless framework and needs much effort to develop this.

ikedam avatar Apr 06 '19 05:04 ikedam

@ikedam @j-rewerts I just released a new version of Gitmask that acts as a true git remote.

It doesn't yet support preserving git commit messages, but I'll be adding support for that in the near future.

AnalogJ avatar Jun 07 '20 05:06 AnalogJ