Skip to content

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

Merged
yasulab merged 2 commits into
masterfrom
update_guides_source_md
Jun 4, 2020
Merged

yasulab merged 2 commits into
masterfrom
update_guides_source_md

Conversation

@hachi8833

@hachi8833 hachi8833 commented May 24, 2020

Copy link
Copy Markdown
Collaborator

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

追記(2020/05/26)

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

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

@yasulab

yasulab commented May 26, 2020

Copy link
Copy Markdown
Member

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

@hachi8833

Copy link
Copy Markdown
Collaborator Author

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


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

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

20200526_155319_E7dpx8
image

@hachi8833

Copy link
Copy Markdown
Collaborator Author

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

@yasulab

yasulab commented May 31, 2020

Copy link
Copy Markdown
Member

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

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

@hachi8833

Copy link
Copy Markdown
Collaborator Author

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

@yasulab

yasulab commented Jun 4, 2020

Copy link
Copy Markdown
Member

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

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

8c68c51b926c2429d602ef8d714477f1

@yasulab yasulab left a comment

Copy link
Copy Markdown
Member

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

yasulab commented Jun 4, 2020

Copy link
Copy Markdown
Member

あとで必要になるかもしれないので、念のため 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