search

haushinka2dx / atmosphere-web

1 stars 2 forks source link

add jasmine for test #12

Closed kannokanno closed 11 years ago

kannokanno commented 11 years ago

これと同様。 https://github.com/haushinka2dx/atmosphere/pull/16

こっちの検討事項ですが、このままだと以下ファイルもデプロイされちゃいますかね。 (ユーザーからアクセス可能かどうか、という意味)

急ぎじゃないので、このあたり決めてからのマージでも問題ないです。 (デプロイ時に該当ファイルは除外する or Nginxの設定でアクセス不可にさせる などなど)

kannokanno commented 11 years ago

ちなみに上記本文下、もしくはMerge pull requestボタンの上に All is well — The Travis CI build passedと出るようになりました。 travisで自動テストが走りOKだったということです。

haushinka2dx commented 11 years ago

確かに。とりあえずはnginx の設定で良いかなと思います。

haushinka2dx commented 11 years ago

これ、こちらでも動いた（ phantom.js 版はまだですが）のでマージしちゃおうかと思いますが何か異論ありますか？ nginx の設定はこれからやります。

kannokanno commented 11 years ago

ないです、お願いします！