@fabiank you pushed https://phabricator.kde.org/D12415 to git usign you as the Author instead of Shlomi.

Are you aware of that? Do you know how to make it so that doesn't happen again in the future and you don't "steal" other people's commits?

About this text, we don't usually credit "authors of other libraries", so If you want to go with Add Author i think the more natural thing would be "Integration with Freecell Solver"

@aacid : Ouch, I wasn't aware of that, and hoped that phabricator would do the right thing. I know how to do this correctly using git, but I have no clue how I get arcanist to preserve the author. Could you point me to the appropriate documentation, so that this mistake doesn't happen again?

In D13809#286309, @fabiank wrote:

@aacid : Ouch, I wasn't aware of that, and hoped that phabricator would do the right thing. I know how to do this correctly using git, but I have no clue how I get arcanist to preserve the author. Could you point me to the appropriate documentation, so that this mistake doesn't happen again?

arc can be a bit annoying. The issue is, people can upload to phabricator patches via two ways, with arc or using the website.

If they use arc, the git information will be collected from their local git commit, you still have to check it's good so that they didn't use something like "I don't care <bla@bla.com>" as their name/email, but most of the times will be good.

If they upload via webpage, there's no git commit, so when you use arc patch to download the patch, arc will use *your* git information as author, so you need to call

git commit --amend --author "Name Surnames <email@domain.extension>"

So it uses the correct author information, if you don't know the author information you'll have to ask him (or sometimes you can ask me too since i have higher access to the identity.kde.org data and i usually can find the name/author from a given nick there)

Does what i said make sense to you?