1
2

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 5 years have passed since last update.

RailsGuides(JP)にPRを出す手順

Last updated at Posted at 2018-09-16

目的

 RailsGuidesを参考にプログラムを動かしてみたら思ったとおりに動作しないときなど、RailsGuidesに問題があったときに修正してPRを出す手順を記載する。

手順

RailsGuidesをGithubでforkする。

 RailsGuidesのPRはforkしたリポジトリでブランチをpushして、本体にPRする流れになる。

スクリーンショット 2018-09-16 10.45.53.png

RailsGuidesをクローンする

$ git clone https://github.com/<forkしたアカウントID>/railsguides.jp.git

RailsGuidesをローカルで起動する

$ bundle install
$ bundle exec rake assets:precompile
$ bundle exec jekyll server

 http://localhost:4000 にアクセスする。

RailsGuidesを編集する

 まずは master ブランチから patch-1 というブランチを切る(数字の1はPR回数が増える毎に加算する。自分がそうしているだけで、もしかしたら正しくないかもしれない)。

$ git branch patch-1 master
$ git checkout patch-1

 エディタで /guides/source/ja 配下の対象ファイル(.md)を探して編集する。
 railsguides.jpで修正したいページをブラウザで開いていれば、URLから対象ファイル名が分かると思うのでファイルはすぐに見つかると思う。

https://railsguides.jp/<ここがmdファイル名と一致する>.html

 編集が完了したら、 add / commit / push する。
 push したら、本体(yasslab/railsguides.jp)のGithubを見るとPRできるようになっているので、PR内容を分かりやすく書いてマージのお願いをする。

PRする前に気をつけること

Rails本家も確認してみる

 誤訳ではなく、コードのミスなどRailsGuide(本家)にも影響がありそうな場合は、既に修正されているかどうか確認してみる。(Railsコミッターになれるチャンス?)
 ただし、本家はEdgeGuidesという最新の修正コミットを適用したページもあるので、そちらで修正されていれば本家は既に修正済みになるので注意が必要。

修正内容をもう一度確認する

 文章を校正するなら本当に読みやすくなっているかとか、コードを修正するなら実際に実行して動作確認を取って間違いがないことを確認する。(当たり前だけど念の為)

実際にやってみたPRリスト

PRリスト 概要
#702 「Active Record コールバック after_touch」の出力結果を修正
#707 「ActionView概要 コレクションを出力する」のコードを修正

更にやってみたこと

PRリスト 概要
rails
#33898
Improve readability in form helpers guide [ci skip]
rails-jp
#712
Improve readability in form helpers guide by white space

参考

RailsGuides日本語訳README
 この記事が古くなったら手順が変わっているかもしれないので、READMEを参考にすることを推奨する。

自分のブログ
 #702 に対してどういう経緯で修正したか、何が問題だったのかを紹介している。

1
2
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
1
2

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?