release blog: generate release post without content #85
Conversation
GitHub: groongaGH-83 In this PR, we implement a Rake task for generating release announces in blog without contents. At the following PRs, we will implement contents step by step. Generate release announces ```console $ VERSION=14.1.1 rake release:blog:generate $ tree . | grep 2024-12-04-groonga-14.1.1.md │ └── 2024-12-04-groonga-14.1.1.md │ │ └── 2024-12-04-groonga-14.1.1.md ```
Because the version is updated when we generate the release posts.
release_task.rb
Outdated
| end | ||
| end | ||
| end | ||
| end | ||
|
|
||
| def post_filename | ||
| "#{Time.now.strftime("%F")}-#{@package.downcase}-#{@version}.md" |
There was a problem hiding this comment.
fix: 25ca18c...2c29500 共通して使いまわせそうだったので、_config.ymlから取得するように修正しました。
release_task.rb
Outdated
| require "yaml" | ||
| require "date" |
There was a problem hiding this comment.
ソートしてください。
| require "yaml" | |
| require "date" | |
| require "date" | |
| require "yaml" |
release_task.rb
Outdated
| jekyll_config["#{@package.downcase}_version"] | ||
| end | ||
|
|
||
| def detect_release_date | ||
| jekyll_config["#{@package.downcase}_release_date"] |
release_task.rb
Outdated
| @@ -28,14 +34,44 @@ def define | |||
|
|
|||
| private | |||
|
|
|||
| def jekyll_config | |||
| YAML.safe_load_file("_config.yml", permitted_classes: [Date]) | |||
There was a problem hiding this comment.
| YAML.safe_load_file("_config.yml", permitted_classes: [Date]) | |
| YAML.safe_load_file(File.join(@jekyll_path, "_config.yml"), permitted_classes: [Date]) |
release_task.rb
Outdated
| end | ||
|
|
||
| def detect_version | ||
| jekyll_config["#{@package.downcase}_version"] |
There was a problem hiding this comment.
なんども@package.downcaseしているのが気になるんですが、initializeでpackageじゃなくてproductあるいはproduct_labelとして受け取って@product_id = product.downcaseして論理的にどういう値として扱っているかを名前でわかるようにするとか、なんとかなりませんか？
There was a problem hiding this comment.
fix: 26072b2 どうするか悩んだのですが、initialize時に受け取った変数を変更せずに持っていたかったので、#product_idメソッドで対応する形にしてみました。（ブログのポストの中で、Groongaで利用するため）
There was a problem hiding this comment.
@versionとかは事前に計算しているので合わせませんか？
@productと@product_idがあってもおかしくないと思います。
There was a problem hiding this comment.
@versionとかをメソッドにする方に寄せるのでもいいと思います。
とにかく揃っていて欲しいです。
There was a problem hiding this comment.
fix: 624c7e5事前に計算する形に揃えました 🙏🏾
We pre-calculate the variables for generating release blog posts.
otegami
force-pushed
the
release-generate-release-posts
branch
from
04be004 to
624c7e5
Compare
kou
changed the title
GitHub: GH-83
In this PR, we implement a Rake task for generating release announces in blog without contents.
At the following PRs, we will implement contents step by step.
Generate release posts