SoftwareFoundationGroupAtKyotoU / automata

Other
3 stars 10 forks source link

master.ymlとlocal.ymlのcheck以下が正しくマージされない。 #57

Closed westpaddy closed 9 years ago

westpaddy commented 9 years ago

正しく(以前)は、check.{defaut,report_id}.{build,test}まで入っていってマージしていたのですが、現在checkをマージするようになっています。

# master.yml
check:
  default:
    build:
      command: "echo hoge"
    test:
      run: "echo hoge"
# local.yml
check:
  default:
    test:
      run: "echo fuga"

これをマージしたときに正しくは

{check: {default: {build: {command: "echo hoge"}, test: {run: "echo fuga"}}}}}

になるはずですが、現在

{check: {default: {test: {run: "echo fuga"}}}}

になっています。

個人的にはlocal.ymlとかなくそう派。

OKU1987 commented 9 years ago

relmとかadmin(メールアドレス)が年度毎に変わるのでlocal.yml使ったほうがいいのかなぁと思ってたんですが、mergeが複雑になっているのなら無くてもいいかもしれないですね。

krtx commented 9 years ago

ということになりました。