search

0918nobita / setup-cljstyle

Set up cljstyle (Clojure code formatter)
MIT License
4 stars 1 forks source link

可読性を上げる #330

Closed 0918nobita closed 3 years ago

0918nobita commented 3 years ago

https://github.com/0918nobita/setup-cljstyle/blob/58f974a8ebd9d0e25124739db6b55348fba57bb6/src/GitHub/RestApi/Releases.purs#L11

https://github.com/0918nobita/setup-cljstyle/blob/58f974a8ebd9d0e25124739db6b55348fba57bb6/src/GitHub/RestApi/Releases.purs#L22-L24

String 型の引数の連続が可読性を下げてしまっているので、ひとつだけ引数に渡すためのレコードを定義してリファクタリングすると良さそう

0918nobita commented 3 years ago

FFI の扱いがイマイチわかってないのでそこらへんも学習しながら進めていく

0918nobita commented 3 years ago

下書き

TypeScript 側の型定義：

type FetchReleaseArgs = {
  authToken: string;
  owner: string;
  repo: string;
};

PureScript 側の型定義：

type FetchReleaseArgs = {
  authToken :: String,
  owner :: String,
  repo :: String
}
0918nobita commented 3 years ago

将来的には node-fetch の binding を提供している purescript-milkis を導入して、TypeScript コードを減らしたい https://github.com/justinwoo/purescript-milkis