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

Adopt rails5 for win #221

Merged
merged 5 commits into from
Jul 18, 2016
Merged

Adopt rails5 for win #221

merged 5 commits into from
Jul 18, 2016

Conversation

igaiga
Copy link
Contributor

@igaiga igaiga commented Jul 17, 2016

  • RailsInstaller3.2.0 で Rails5 環境を作る手順で更新

  • node.js の記述が古かったので更新

  • 動作確認をnode.jsのインストール後にやった方がいいので順番入れ替え

  • WinでもRails5の環境構築をする内容になったので、rake db:migrate → rails db:migrateへ

    • appページでOS依存文がなくなったので削除

    RailsInstaller3.2.0 で Rails5 環境つくりの詳細はこちらのPR参照 : 'rails db:migrate' is recommended on Rails 5 #203

@igaiga
Copy link
Contributor Author

igaiga commented Jul 17, 2016

@yuumi3 こちら確認お願いできますか?

@yuumi3
Copy link
Contributor

yuumi3 commented Jul 17, 2016

@igaigaさん、おつかれさまです。
「Rails Girls アプリ・チュートリアル」に Gemfileの gem 'execs' を追加する手順が抜けてると思います。

@igaiga
Copy link
Contributor Author

igaiga commented Jul 18, 2016

@yuumi3 execjs ですよね?
Gemfileに明に書かなくても、Gemfile.lockにexecjs入っていたので、書かなくても大丈夫じゃないかと思ってます。(勘違いだったら教えてください)

作られたGemfile.lock

coffee-script (2.4.1)
  coffee-script-source
  execjs
...
execjs (2.7.0)

@yuumi3
Copy link
Contributor

yuumi3 commented Jul 18, 2016

execs をGemfileに指定指定しないとエラーになりましたが、家に戻ったら試してみます。

@yuumi3
Copy link
Contributor

yuumi3 commented Jul 18, 2016

VMを巻き戻して最初から実行したら出来ました。

@igaiga
Copy link
Contributor Author

igaiga commented Jul 18, 2016

yuumi3さん確認ありがとうございました。では、マージしますね。

@igaiga igaiga merged commit 2e89baf into master Jul 18, 2016
@urimaro urimaro deleted the adopt_rails5_for_win_on_install_page branch May 19, 2023 04:11
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