Closed shinsuke-mat closed 4 years ago
resolve #751
Exporter と Patch の2つのクラス周辺をリファクタリング. Serializer 関係は触らず.
Exporter
Patch
Serializer
存在意義のないExporterFactoryの削除 Exporters extends Export の排除 ファイル出力関係のテスト(のうちPatchとExport周り)にJimfsを導入 無駄なロガー削除 immutabilityが皆無なPatch周りを改善 Patches削除 出力ディレクトリ周りの操作を改善 テストの可読性を確保 ...
Exporters extends Export
ほぼゼロから実装 & 差が多すぎるので,diff見るよりソース単体を見た方が良い.
review ありがとう.反映しました. 3点議論必要です.
お疲れさまです. マージします
resolve #751
Exporter
とPatch
の2つのクラス周辺をリファクタリング.Serializer
関係は触らず.やったこと(抜粋)
存在意義のないExporterFactoryの削除
Exporters extends Export
の排除 ファイル出力関係のテスト(のうちPatchとExport周り)にJimfsを導入 無駄なロガー削除 immutabilityが皆無なPatch周りを改善 Patches削除 出力ディレクトリ周りの操作を改善 テストの可読性を確保 ...review
ほぼゼロから実装 & 差が多すぎるので,diff見るよりソース単体を見た方が良い.
752 と確実にコンフリクトするので,ひとまず内容だけ確認してほしい.