holiday_jp-go再始動
k1LoW opened this issue · comments
@holiday-jp/maintainers
私もGoが書けるようになったので、holiday_jp-goも最新の祝日に追従させていきたいと思っています。
#4 というような素晴らしい提案もあるので、うまく取り入れられるようにしたいです。
今のところ実施したいことは以下です。
- Go Modules対応
- YAMLを読み込む形ではなく、YAMLからGoコードをジェネレートする形にする(statikからの脱却)
- GitHub Actions対応
もし何かご意見あればよろしくお願いいたします。
2020/12/10くらいから作業をしたいと思います。
go 1.16 から statik じゃなくても embed できるようになるので、そうなったら(来年2月頃)また PR を出そうと考えていたところでした
@okitan 反応できていなくてすみません!先ほどActions対応のPRを取り込みました!ありがとうございます!
go 1.16 から statik じゃなくても embed できるようになるので
YAMLをembedして読むよりYAMLからGoのコードを生成したほうがYAMLを読み込む処理がなくなる分パフォーマンス的にも良いと考えていますがいかがでしょう?
取り急ぎ最新のデータセット(v1.3.0)に追従しました
パフォーマンスとしては generate するほうが良いでしょうね(どちらにしてもそんなに気にする程度ではないと思います)
コードを生成する分仕組みが複雑になるのがデメリットですが、他の言語に比べgoはgenerateすることが一般的ですし、特に懸念はないです
どちらにしてもそんなに気にする程度ではないと思います
おっしゃる通りですねえ。。
YAMLを抱えてもgenerateしてもあまり差がないという点に同意なのでここでCloseします