Integrate Wercker CI #28
Conversation
wercker.yml
Outdated
| - script: | ||
| name: pip install -r | ||
| code: | | ||
| pip install -r requirements.txt |
There was a problem hiding this comment.
この script 部分は、次のステップの pip-install で同じコマンドが実行されているようなので、不要だと思います。
There was a problem hiding this comment.
あとそもそも、setup.pyでrequirements.txtの中身がinstall_requiresに加えられているので、pip install -r requirements.txt をする必要はないかと
pip install -r requirements
pip install .
ではなく、
pip install .
の一コマンドで依存関係もまとめてインストールできるはずです。
There was a problem hiding this comment.
コメントthanks. wercker上でのpipの挙動が少し謎で，requirementsに記載された，pysptkをnumpyより先にインストールしようとし，pysptkのインストール時にnumpyがないと怒られる状況だったので上記の様に記述していました．pip install .でsetup.pyが走る事を知りませんでした．ありがとうございます．
There was a problem hiding this comment.
150a312 どうやら，pipのインストールがコケてるみたいです．
There was a problem hiding this comment.
https://github.com/wercker/step-pip-install 参考まで
pip install .でsetup.pyが走る事を知りませんでした．
これはそうなのですが、pysptkのsetup.pyが走るときにnumpyは必要なので、numpyを事前にインストールする必要があるのは変わりません
There was a problem hiding this comment.
なるほど．どうやら
- pip-install:
clean_wheel_dir: true
でも，pip install . が走るようなので，先程修正した部分を省略しました．
modify wercker.yml to install nosetests before testing ignore .wercker directory for staging
|
マージします． thanks for review. |
Werckerを導入した．python2環境の自動テストが動作する事を確認した．python3への対応 #4 および ドキュメント化とテストコードの作成 #27 は，リンクのブランチで実施中．両者の変更時に，wercker.ymlの追記・編集していくのが良いかと考えてます．