-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Fix & Add: Japanese Translation (6) #3304
Conversation
If nothing major seems to be wrong until today night, I will ask pfrazee-san to merge it. |
mainブランチに入った最近の変更の多くが巻き戻ってしまっているようです。このままマージすることはできません。Files changedを見て、 コンフリクトが発生するたびにPRを作り直しているようですが、GitHubのヘルプにある手順ではコンフリクトを解決できなかったのでしょうか? test.txtを作ったり消したり、messages.poを一度消してから上げ直したりと意味のないコミットが入っているのも気になります。 |
@tkusano @dolciss @oboenikui @noritada @middlingphys @hibiki @reindex-ot @haoyayoi @vyv03354 44840ac で新しいバージョンに対応させました。 こちらの都合で申し訳ないですが、数日後から忙しくなるため、ファイルの場所の指定など致命的なミスがない場合は、一回mainブランチの方にマージを依頼します。 その後、どなたでもいいので主体となって進めていただけると嬉しいです。 |
実行環境を持っておらず、Webから操作できる方法でやっているためです。 意味がないとありますが、私にとっては上記環境でやるために工数を減らす(txtを作っているのは、一番下のディレクトリにあるファイルを消すとディレクトリが吹っ飛ぶのでそれを避けるため、アップロードしているのは、Webから操作する際にこれまでの操作を引き継ぐため)ための意味がある変更だと考えております。 個人的にはもう少しフローを改善したいと思っていたため、教えていただいたリファレンスを後日読もうかと思っています。 貴重なご意見ありがとうございました。 |
レビューは後で確認します。 |
GitHubのヘルプにある手順はwebから操作できる方法です。[Resolve conflicts]ボタンが表示されなかったのでしょうか? |
@pfrazee |
Thank you for all the changes and merges.
It seems like they want it to be merged quickly, so I did a easily check and it looks good to me. |
@ tkusano thanks. |
Confimed that the messages I pointed out is fixed. Ready to merge. |
@pfrazee (as a remind) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing! Thank you everybody!
@tkusano @dolciss @oboenikui @noritada @middlingphys @hibiki @reindex-ot @haoyayoi @vyv03354
Please review this PR.
This PR includes the following changes.