-
Notifications
You must be signed in to change notification settings - Fork 180
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
Translate glossary #21
Conversation
@smikitky crowdin の時は翻訳されていなかったみたいですが、ソースコードのコメントも翻訳するかどうかは追加で検討したほうがですね 🤔 |
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.
よろしくお願いします
サジェスチョンを1個1個コミットすると大量のコミットが作成されるのでバッチでコミットする機能を使ってみてください |
今帰宅中なので後で見ますー。 |
少なくともこのPRに関していえば squash で問題ありませんが、他のPRに関してもマージバブル作るべきかどうかについては、確かに考えないとですね… |
Co-Authored-By: potato4d <mail@potato4d.me>
@smikitky 対応したので再度見ていただけると。マージについてですが、 "typo" みたいなコミットメッセージが入ってしまっても微妙ですし、おりをみて全て squash merge に限定しても良いかなぁと思っています。 特に、英語ドキュメントで新規の翻訳が出た場合、翻訳レポジトリでは 1 コミット目に英文を追加して続くコミットで diff を取りたくなることが多々あるので、 Pull Request 時点ではコミットが多いほうが円滑な気がしました(最悪 squash merge しなくても rebase すれば良いので) |
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.
いくつかフォーマットの修正と単語の修正(数字 -> 数値)の確認をお願いします。
mergeに関しては、 squash merge に限定するのに賛成です。
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.
もう少しだけ細かい修正点を見つけましたので対応お願いします
Co-Authored-By: potato4d <mail@potato4d.me>
Co-Authored-By: potato4d <mail@potato4d.me>
@smikitky 良きときにマージを貰えるとー🙏 |
This pull request resolve #2