Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Discusion] Localize Commit Message for PullRequest Update #9823

Closed
wants to merge 1 commit into from

Conversation

6543
Copy link
Member

@6543 6543 commented Jan 17, 2020

come up in #9784

question is Yes/No

@6543
Copy link
Member Author

6543 commented Jan 17, 2020

I personaly dont like to localice ... reason:

Lunny has set the gitea language to: Chinese ...
Lafriks to Latvian ...
Me to German ...

If we update a branch no one knows what is going on!!!

think thats why github use englisch only for the update function too

@guillep2k ⬆️

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 17, 2020
@guillep2k
Copy link
Member

We solved this problem for other resources (like WIP, closes, fixes, etc.) by setting up a value in app.ini, so site admins can tailor it to a better suited language in their case. Ultimately, these options should also be configurable per repo or per repo-owner, but site-wide is more than enough for now.

I agree that this is not a job for Crowdin.

@6543
Copy link
Member Author

6543 commented Jan 17, 2020

@guillep2k I'm for a repo setting if we like to locale

@lunny
Copy link
Member

lunny commented Jan 17, 2020

A per repository setting maybe better.

@lafriks
Copy link
Member

lafriks commented Jan 17, 2020

It can be app.ini setting as default and changeable by org/repo settings

@guillep2k
Copy link
Member

guillep2k commented Jan 17, 2020

Let's vote:

  • app.ini: 👍
  • by repository, then by app.ini: 😄
  • by owner (e.g. all of go-gitea), then by app.ini: 👀
  • by repository, then by owner (e.g. all of go-gitea), then by app.ini: ❤️
  • Via Crowdin: 🚀
  • Nothing at all: 👎

More than one vote is valid.

@lafriks lafriks added the type/enhancement An improvement of existing functionality label Jan 17, 2020
@lafriks
Copy link
Member

lafriks commented Jan 17, 2020

@guillep2k it can be both changeable in org settings and per-repo

@davidsvantesson
Copy link
Contributor

davidsvantesson commented Jan 17, 2020

I would suggest .md files (or other text files) in the repository (like the PR template)
That is something I have though about also for the PR merge commit message.

It has to support replacable variables, like [target-branch] and similar (that should be true for any solution.)
Markdown files would make it possible to have one for whole site and one in the repo.

Localized to the current user is not a good idea imho.

@lafriks
Copy link
Member

lafriks commented Jan 17, 2020

@davidsvantesson not current user but repo owner

@davidsvantesson
Copy link
Contributor

That would only work for user repos? Would you add a language for organizations as well?
I think many would not want their commit messages translated just because they choosed to have the UI in a specific language. Many (or most?) organizations have a policy to only use english.

@guillep2k
Copy link
Member

That would only work for user repos? Would you add a language for organizations as well?
I think many would not want their commit messages translated just because they choosed to have the UI in a specific language. Many (or most?) organizations have a policy to only use english.

We seem to be keeping two different discussions here, and I'm not sure what are we talking about anymore 🤔. In my mind, this issue was about customizing the commit message for updating a PR, and that only. And by customizing I mean type in any text you want, not choosing a locale. It seems, however that others are thinking of using the locale data and customize which language will be used for each org/repo, which of course is a different matter.

Anyone want to add/clarify about their position? 😬 We may need another vote.

@6543
Copy link
Member Author

6543 commented Jan 17, 2020

I think discusion was if commit-message shuld be localiced ... and the towards solution was: No but changable via settings (so it is manualy localicable via settings)

@davidsvantesson
Copy link
Contributor

@guillep2k You mean that when pressing Update branch you get option to change the commit message? I think that should be a drop-down option in that case to avoid unnecessary clicking. You can only use the Update branch button if there are no merge conflicts, and I don't see why you would to alter the commit message at different occasions.

@techknowlogick techknowlogick changed the title [Discusion] Localice Commit Message for PullRequest Update [Discusion] Localize Commit Message for PullRequest Update Jan 17, 2020
@guillep2k
Copy link
Member

@guillep2k You mean that when pressing Update branch you get option to change the commit message? I think that should be a drop-down option in that case to avoid unnecessary clicking. You can only use the Update branch button if there are no merge conflicts, and I don't see why you would to alter the commit message at different occasions.

Ha ha ha, no, that would be yet another possibility 🤣. I meant it to be a setting at the site level (app.ini), user level (owner, applying to all of their repositories) or repository level.

@davidsvantesson
Copy link
Contributor

I think there are two main alternatives discussed here:

  • Localized using crowdin (by some user's or even org's preferred language)
  • A setting that can be made at different levels.

It seems not many are in favour of the first alterantive. For the settings alternatives, I don't see them as opposing each other. If it is first added as a app.ini setting, someone can then make a new PR with a repository setting (that would take precedence) and then even an organization setting (with in-between precedence).

@guillep2k
Copy link
Member

If it is first added as a app.ini setting, someone can then make a new PR with a repository setting (that would take precedence) and then even an organization setting (with in-between precedence).

I really like that plan. 👍

@6543
Copy link
Member Author

6543 commented Jan 17, 2020

I would sugesst to use placeholders for branch name so they could be added removed and orderd head or base branch first ...

@sapk
Copy link
Member

sapk commented Jan 18, 2020

We could also manage a template for the merge commit in the same way.

@6543 6543 closed this Jan 22, 2020
@6543 6543 deleted the localice-pull-update-commit branch January 22, 2020 09:12
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants