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

WinでRailsInstaller3.3.0 の問題を修正する記述を追加 #291

Closed
wants to merge 1 commit into from

Conversation

igaiga
Copy link
Contributor

@igaiga igaiga commented Jun 1, 2017

installページのWinのRailsInstallerに関する問題を解決する記述を追加します。

  • (ついでに)RailsInstallerを英語ガイドにあわせて3.3.0へ
  • RailsInstaller3.3.0用のバグ修正方法を追記

英語版ガイドのこのPRの日本語版です。 railsgirls/guides.railsgirls.com#338

@sho-h
Copy link
Member

sho-h commented Jun 1, 2017

3.2.1(Ruby 2.2系)だとこの作業なしでも動くようでしたけど、3.3.0(Ruby 2.3系)じゃないとRails 5系では動かないとかでした?

@igaiga
Copy link
Contributor Author

igaiga commented Jun 3, 2017

@sho-h RailsInstaller3.2.1(Ruby 2.2系) + Rails5.1は試してないです。
RailsInstaller3.2.0も同様のインストーラバグあったので3.2.1もダメだと思ってたのですが、3.2.1だと実は直ってます?
3.2.0のバグはこれ: railsinstaller/railsinstaller-windows#76

  • 3.2.0→×(バグ有り)
  • 3.2.1→?
  • 3.3.0→×(バグ有り)

@sho-h
Copy link
Member

sho-h commented Jun 5, 2017

@igaiga 僕もRails5.1との組み合わせでは試してないのですが、このPRで直ってる問題なら3.2.1だと大丈夫じゃないかと。5.0系は問題なかったと思います。前見た時は「3.3.0でもリリースしたよ」って言ってた気がするのですが、3.3.0では引き続き同じ問題が出てました。3.2.0までしかタグがないから動かしてみないとわからなくてつらいのがアレなんですけど。

直りきってなくて、Rails5.1をインストールした時にまた動かなくなるようならマージするのがいいんじゃないかと思います。:)

@igaiga
Copy link
Contributor Author

igaiga commented Jun 6, 2017

@sho-h お、なんか直ってるぽい〜!3.2.1で動作確認してみますね。
RailsInstaller3.2.1+Rails5.1でインストーラ問題が解決&RailsGirlsのアプリが動くようなら、このまま3.2.1にしておいて、3.3系が直ったら更新するのが良さそうですね。(´▽`)

@igaiga
Copy link
Contributor Author

igaiga commented Jun 8, 2017

@sho-h RailsInstaller3.2.1(Ruby2.2.6)+Rails5.1.1 で/appと/herokuのページがちゃんと動きました!大丈夫そうです。:)

ということで現状のガイドのままで問題ないので、このPRは閉じますね。
ご指摘ありがとうございました! ✨

@igaiga igaiga closed this Jun 8, 2017
@sho-h
Copy link
Member

sho-h commented Jun 9, 2017

@igaiga おぉ、よかったです!確認ありがとうございました!

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