crowdin / github-action

A GitHub action to manage and synchronize localization resources with your Crowdin project
https://crowdin.com
MIT License
151 stars 52 forks source link

Add translators to Co-Authored-By #35

Open TurnrDev opened 3 years ago

TurnrDev commented 3 years ago

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.

jesec commented 3 years ago

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.

NickVolynkin commented 3 years ago

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.

TurnrDev commented 3 years ago

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?

uiolee commented 1 year ago

agree

rickstaa commented 9 months ago

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.

anatawa12 commented 6 months ago

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.