SoftwareFoundationGroupAtKyotoU / automata

Other
3 stars 10 forks source link

Separated functions concerning configure files from app.rb #59

Closed westpaddy closed 9 years ago

westpaddy commented 9 years ago

confの機能分離集約しました。

krtx commented 9 years ago

という感じです(あんまりちゃんと見てなくて……ごめんなさいm(._.)m)

westpaddy commented 9 years ago

local.yml がないとエラーになるのはバグです。 configファイル(master,scheme,template)へのアクセスをconfオブジェクト経由に集約しました。confオブジェクト自体はほぼハッシュで:master,:scheme,:template以下に各ファイルをロードした値が入ってます。 たとえば、master.yml の logger.path には conf[:master, :logger, :path] でアクセスできます。

westpaddy commented 9 years ago

local.yml の件は直しました。

krtx commented 9 years ago

local.yml がなくても動くことは確認できました。

各種スクリプトはapp経由でconfにアクセスしていますが、それも分離するというのは可能でしょうか? (各ファイルの中で conf インスタンスを作って設定を読み込む)

westpaddy commented 9 years ago

できますが、そっちのほうがいいかどうかはわからないですね

westpaddy commented 9 years ago

conf のインスタンス直接作れるようにする場合 config ディレクトリのパスはどこに持たせるのがいいですか?

chiro commented 9 years ago

app.rbかconf.rbならどちらでもよさそうだけど、個人的にはconf.rbです。

krtx commented 9 years ago

app から conf の機能を分離するということだったら、conf.rb でしょうか

chiro commented 9 years ago

67 で取り込まれました。