search

BrainPad / cliboa

application framework for ETL(ELT) pipeline, process
MIT License
20 stars 13 forks source link

[〆6/7] move http from util to adapter #426

Closed cocoa-maemae closed 1 year ago

cocoa-maemae commented 1 year ago

Brief

http.pyをutilからadapterに移動 テストコードも同時に移動

Points to Check

Test

Confirmed

（Write content to confirm / Reason why not necessary）

Review Limit

yasuhiro-ohba commented 1 year ago

すみません、http系のメソッドを追加したためutil周りも手を入れました。 お手数ですが、再度変更部分含めたリファクタリング対応いただけると嬉しいです。

cocoa-maemae commented 1 year ago

すみません、http系のメソッドを追加したためutil周りも手を入れました。 お手数ですが、再度変更部分含めたリファクタリング対応いただけると嬉しいです。

承知しました。対応します。

cocoa-maemae commented 1 year ago

@yasuhiro-ohba こちらのコンフリクトの解決を試みましたが、認識合わせが必要そうです。 元々util/http.pyで実装していた内容と一部同じ処理(http GET,POSTなど)をconnect/http_request.pyで再度実装したという理解で合ってますでしょうか？

このチケットではutil/http.pyをadapter/http.pyに移行しようとしていたので、connect/http_request.pyもadapter/http.pyに統合した方がいいと思いました。

アプリケーションの階層の数と依存関係はこれまで通りinterface>core>scenario>adapter>utilの5つのままにしたいと思います。（階層増やしてしまうと依存関係がよくわからなくなってしまうため）

yasuhiro-ohba commented 1 year ago

@cocoa-maemae connectの階層を削除しましたので、再度コンフリクトを解消いただければと思います。

cocoa-maemae commented 1 year ago

@cocoa-maemae こちら遅くなりましたが対応完了しました。