github-action icon indicating copy to clipboard operation
github-action copied to clipboard

Add translators to Co-Authored-By

Open TurnrDev opened this issue 5 years ago • 7 comments

Is your feature request related to a problem? Please describe. When a translation PR comes through, it's attributed to just Crowdin Bot, which is a tad unfair on the actual translators.

Describe the solution you'd like Use Co-Authored-By to attribute the actual people who worked on the translations.

Describe alternatives you've considered Using the Last-Translator flag on the file, but that only attributes the latest author

Additional context Translators are just as important to a project as a Developer is, by making use of the Co-Authored-By feature in git, we can give them some credit for their work.

TurnrDev avatar Oct 01 '20 15:10 TurnrDev

Totally agree.

Weblate already has the feature: https://github.com/firewalld/firewalld/pull/697, https://github.com/Radarr/Radarr/commit/aafd82ba739541f1a48ba20c3bf13b7ffb8d94f6.

Translators from the community are entitled to be properly recognized and attributed. Without such a feature, a human would be needed to lookup, review and compile a list of contributors, which is time-consuming and slows the translation acceptance process.

jesec avatar Jan 29 '21 12:01 jesec

The problem here is that we need to know the translators' emails that they've added to their GitHub accounts. These can be different from the emails known to Crowdin. Also, this is a kind of personal info disclosure, so we'll need to ask for permission to put their name & email in the commit message and then store it somewhere in Crowdin.

Anyway, I'll be happy to see this feature at least in the Crowdin API and CLI.

NickVolynkin avatar May 12 '21 12:05 NickVolynkin

The problem here is that we need to know the translators' emails that they've added to their GitHub accounts

Not a problem, just use the one they use for Crowdin. It's about attribution, not GitHub itself.

Also, this is a kind of personal info disclosure, so we'll need to ask for permission to put their name & email in the commit message

I mean, a generic "Allow the project manager to attribute my name and email on external services" setting on Crowdin would be good. I'm sure most project managers want to attribute translators anyway?

TurnrDev avatar May 27 '21 20:05 TurnrDev

agree

uiolee avatar Sep 07 '23 09:09 uiolee

Having this feature would significantly enhance user interaction 🚀. When receiving translations, I ask the translators to provide their GitHub usernames in the chat. However, we could streamline this process by implementing an opt-in system where users link their GitHub accounts. If a GitHub account is connected, it could automatically include their account as a co-author.

It's worth noting that I haven't explored whether their GitHub email addresses are accessible to third-party services in this manner. Typically, I retrieve this information by adding the .patch suffix to one of the user's commits, such as this commit.

rickstaa avatar Dec 13 '23 08:12 rickstaa

I think it's better to use Author attribute of git commit if there is only one related author since it's git-builtin. In git, we can specify different Author than Committer so I think it's good to specify bot as a committer and author as translator.

anatawa12 avatar Mar 09 '24 10:03 anatawa12