-
-
Notifications
You must be signed in to change notification settings - Fork 276
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
ガイドのソースを6.0時点のものに更新する #938
Conversation
@hachi8833 念のためどのコミット時点を基準としてアップデートしたのか記録できると嬉しいです!どのコミット時点までアップデートされましたか? 👀💭 |
@yasulab 冒頭のコミットメッセージに取り込み元の情報を追記しました🙇。 もう一点、security.mdのみ、ソースに記号が生書きされていてGitlocalizeでエラーが発生したので、このファイルのみやむを得ず修正しています(添付スクショ参照)。 このソースの修正は rails/rails#38610 でRailsのmasterブランチにマージされているので、今後6.1を取り込むときにsecurity.mdだけ注意が必要になります。 |
@yasulab 原文のsecurity.mdの独自修正について考え直し、66cabeda2cと同じ状態に戻しました。これで原文のコミットが揃いました。 |
@hachi8833 ご対応ありがとうございます!diff の量が多いので、可能であれば原文の変更 PR と訳文の変更 PR を分けてもらえると助かるのですが、そういった対応って可能でしょうか? 👀 💭
|
|
@hachi8833 地道ですが GitHub の折りたたみ機能を使えば1ファイルずつレビューできそうなので (英文ファイルを折り畳んで訳文ファイルをレビューする方針でできそうなので)、こちら僕の方で進めてみますね 👀💨 |
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.
あとで必要になるかもしれないので、念のため tag も作っておきました!d( ̄  ̄)✨ |
#937 に関連して、最初にソースmdだけ更新します。マージいただいた後、訳文mdファイルごとに順次プルリクを作成します。
追記(2020/05/26)
@yasulab 以下の情報を追記します🙇