Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

holiday_jp-go再始動 #6

Closed
2 of 3 tasks
k1LoW opened this issue Dec 6, 2020 · 6 comments
Closed
2 of 3 tasks

holiday_jp-go再始動 #6

k1LoW opened this issue Dec 6, 2020 · 6 comments
Assignees

Comments

@k1LoW
Copy link
Member

k1LoW commented Dec 6, 2020

@holiday-jp/maintainers

私もGoが書けるようになったので、holiday_jp-goも最新の祝日に追従させていきたいと思っています。

#4 というような素晴らしい提案もあるので、うまく取り入れられるようにしたいです。

今のところ実施したいことは以下です。

  • Go Modules対応
  • YAMLを読み込む形ではなく、YAMLからGoコードをジェネレートする形にする(statikからの脱却)
  • GitHub Actions対応

もし何かご意見あればよろしくお願いいたします。
2020/12/10くらいから作業をしたいと思います。

@k1LoW k1LoW self-assigned this Dec 6, 2020
@okitan
Copy link
Contributor

okitan commented Dec 20, 2020

go 1.16 から statik じゃなくても embed できるようになるので、そうなったら(来年2月頃)また PR を出そうと考えていたところでした

@k1LoW k1LoW mentioned this issue Dec 20, 2020
@k1LoW
Copy link
Member Author

k1LoW commented Dec 20, 2020

@okitan 反応できていなくてすみません!先ほどActions対応のPRを取り込みました!ありがとうございます!

go 1.16 から statik じゃなくても embed できるようになるので

YAMLをembedして読むよりYAMLからGoのコードを生成したほうがYAMLを読み込む処理がなくなる分パフォーマンス的にも良いと考えていますがいかがでしょう?

@k1LoW
Copy link
Member Author

k1LoW commented Dec 20, 2020

取り急ぎ最新のデータセット(v1.3.0)に追従しました

@okitan
Copy link
Contributor

okitan commented Dec 20, 2020

パフォーマンスとしては generate するほうが良いでしょうね(どちらにしてもそんなに気にする程度ではないと思います)
コードを生成する分仕組みが複雑になるのがデメリットですが、他の言語に比べgoはgenerateすることが一般的ですし、特に懸念はないです

@k1LoW
Copy link
Member Author

k1LoW commented Dec 21, 2020

どちらにしてもそんなに気にする程度ではないと思います

おっしゃる通りですねえ。。

@k1LoW
Copy link
Member Author

k1LoW commented Dec 26, 2020

YAMLを抱えてもgenerateしてもあまり差がないという点に同意なのでここでCloseします

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants