zunda-cafe / zunda-devstudy

イマドキの開発手法を実践しながらプロダクトを作成する
0 stars 0 forks source link

リファクタリング(pomの細かい調整、不要ソース削除) #13

Closed maruhachi closed 7 years ago

maruhachi commented 7 years ago

やったこと

12 のチェックボックスにある内容はこっちのコミット にて修正しています。

10 のtargetフォルダ削除をこっちのコミット で実施しています。

ちなみに 先日、IntelliJからのGitPushで新しく作ったブランチへpushするはずがdevelop(枝葉元)へpushされた、という事象があったと思います。 これはpushダイアログのorigin側ブランチ名を編集すればよかったようです intellij_git_push

このPRで期待する結果

関連issue

10, #12

ブランチの分岐元を間違えたため、先程pushしたブランチは無視してください。(もう消してます)

naninuneter55 commented 7 years ago

私の宿題(devtoolのscopeを変えるとか、application.propertiesファイルのthyme leafのcache設定は不要だとか)まで修正してもらって、ありがとうございます。 ところで、devtoolのscopeはtestに変更するという話だったと思いますが、runtimeに変更したのはどのような理由からでしょうか?

maruhachi commented 7 years ago

@kazutxt このPRはソースコードには手を入れない方針で実施しました。 プリセット値が入っちゃってた部分は、ソースのリファクタリングの中で残課題としてやっていこうと思ってました。

過半数(2人)のapproveを得られたのでマージします :smiley: