Weekly PullReq #19 - Eukleides project

今週は3件といっても1件にずっと出突っ張りだった。

kubevirtというkubernetesでvirtual machine management tool用のkubernetesデータインポーターであるcontainerized-data-importer

のloggerをglogからklogに入れ替えました。

glogからklogへの入れ替えは以前にチャレンジして失敗して、その時にflagが初期化されてしまうので、フラグを移し替える処理を入れなければならないことを覚えていたので、そこだけ注意しました

ところが、mainメソッド内だけでいいものかと思いきや、至る所でflag.Parse()とレビュアーに気づかされ、また折れかけましたがなんとか実装しました。コードがイマイチ汚い感じになってしまいましたが…。

改修がバンバン入ってrebaseのオンパレードの中、改修範囲が広いのでconflictしまくりで大変でした…orz

他の2件は、assetファイル管理ツール？なのかなpinagoladaというツールのtestのoutput抑制とversion表示の改行を入れました。