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

ガイドのソースを6.0時点のものに更新する #938

Merged
merged 2 commits into from
Jun 4, 2020

Conversation

hachi8833
Copy link
Collaborator

@hachi8833 hachi8833 commented May 24, 2020

#937 に関連して、最初にソースmdだけ更新します。マージいただいた後、訳文mdファイルごとに順次プルリクを作成します。

追記(2020/05/26)

@yasulab 以下の情報を追記します🙇

  • 引用元タグ: v6.0.0
  • 引用元コミット: 66cabeda2c
  • gitlocalize_jpブランチにソースを取り込んだ時のコミット: 330f2e7

@yasulab
Copy link
Member

yasulab commented May 26, 2020

@hachi8833 念のためどのコミット時点を基準としてアップデートしたのか記録できると嬉しいです!どのコミット時点までアップデートされましたか? 👀💭

@hachi8833
Copy link
Collaborator Author

@yasulab 冒頭のコミットメッセージに取り込み元の情報を追記しました🙇。


もう一点、security.mdのみ、ソースに記号が生書きされていてGitlocalizeでエラーが発生したので、このファイルのみやむを得ず修正しています(添付スクショ参照)。

このソースの修正は rails/rails#38610 でRailsのmasterブランチにマージされているので、今後6.1を取り込むときにsecurity.mdだけ注意が必要になります。

20200526_155319_E7dpx8
image

@hachi8833
Copy link
Collaborator Author

@yasulab 原文のsecurity.mdの独自修正について考え直し、66cabeda2cと同じ状態に戻しました。これで原文のコミットが揃いました。
原文のsecurity.mdは、訳文のsecurity.mdと同時にプルリクを投げることにします🙇。

@yasulab
Copy link
Member

yasulab commented May 31, 2020

@hachi8833 ご対応ありがとうございます!diff の量が多いので、可能であれば原文の変更 PR と訳文の変更 PR を分けてもらえると助かるのですが、そういった対応って可能でしょうか? 👀 💭

原文のsecurity.mdは、訳文のsecurity.mdと同時にプルリクを投げることにします🙇。

@hachi8833
Copy link
Collaborator Author

diff の量が多いので、可能であれば原文の変更 PR と訳文の変更 PR を分けてもらえると助かるのですが、そういった対応って可能でしょうか? 👀 💭
@yasulab 了解です!security.mdのPRは原文と訳文を分けます。

@yasulab
Copy link
Member

yasulab commented Jun 4, 2020

diff の量が多いので、可能であれば原文の変更 PR と訳文の変更 PR を分けてもらえると助かるのですが、そういった対応って可能でしょうか? 👀 💭

@hachi8833 地道ですが GitHub の折りたたみ機能を使えば1ファイルずつレビューできそうなので (英文ファイルを折り畳んで訳文ファイルをレビューする方針でできそうなので)、こちら僕の方で進めてみますね 👀💨

8c68c51b926c2429d602ef8d714477f1

Copy link
Member

@yasulab yasulab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

一通り確認しました! source/ja 以下のファイルは本 PR に含まれていなかったようなので、一旦こちらでマージしますね 🔧💨✨

image

@yasulab yasulab merged commit 4d75e74 into master Jun 4, 2020
@yasulab yasulab deleted the update_guides_source_md branch June 4, 2020 02:40
@yasulab
Copy link
Member

yasulab commented Jun 4, 2020

あとで必要になるかもしれないので、念のため tag も作っておきました!d( ̄  ̄)✨
https://github.com/yasslab/railsguides.jp/tags

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants