-
Notifications
You must be signed in to change notification settings - Fork 16
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
release blog: generate release post contents #87
Conversation
GitHub: groongaGH-83 In this PR, we implement a Rake task for generating release announce posts in blog. Generate release announce posts ```console $ rake release:blog:generate $ tree . | grep 2024-11-15-groonga-14.1.0.md │ └── 2024-11-15-groonga-14.1.0.md │ │ └── 2024-11-15-groonga-14.1.0.md ``` ```console $ cat en/_posts/2024-11-15-groonga-14.1.0.md --- layout: post.en title: Groonga 14.1.0 has been released description: Groonga 14.1.0 has been released! --- Groonga 14.1.0 has been released! For installation instructions on your environments, please see the [Installation Guide](/docs/install.html). For the information on the changes, please see the [Release Note](/docs/news/14.html#release-14-1-0-2024-12-04). $ cat ja/_posts/2024-11-15-groonga-14.1.0.md cat ja/_posts/2024-11-05-groonga-14.1.0.md --- layout: post.ja title: Groonga 14.1.0リリース description: Groonga 14.1.0をリリースしました! --- Groonga 14.1.0をリリースしました! それぞれの環境毎のインストール方法は、[インストール](/ja/docs/install.html)をご確認ください。 主な変更点のついては、[リリースノート](/ja/docs/news/14.html#release-14-1-0)をご確認ください。 ```
8f49ea3
to
9502f05
Compare
release_task.rb
Outdated
|
||
#{@product} #{@version}をリリースしました! | ||
|
||
それぞれの環境毎のインストール方法は、[インストール](/ja/docs/install.html)をご確認ください。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
様々な @product
で利用する前提の作りだと思いますが、 /ja/docs/install.html
このパスは共通なんでしたっけ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GroongaとMroongaは共通なのですが、PGroongaに関しては共通ではないですmm
メソッドに切り出して、overrideしやすいようにしておいたほうが良さそうですね!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix: afe8102
release_task.rb
Outdated
|
||
それぞれの環境毎のインストール方法は、[インストール](/ja#{product_install_url})をご確認ください。 | ||
|
||
主な変更点のついては、[リリースノート](/ja#{product_release_note_url})をご確認ください。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
主な変更点のついては、[リリースノート](/ja#{product_release_note_url})をご確認ください。 | |
主な変更点は[リリースノート](/ja#{product_release_note_url})をご確認ください。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix: 007789d
release_task.rb
Outdated
case locale | ||
when "ja" | ||
post_ja_content | ||
when "en" | ||
post_en_content | ||
else | ||
raise "#{locale} isn't supported for release announce posts in blog." | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
case locale | |
when "ja" | |
post_ja_content | |
when "en" | |
post_en_content | |
else | |
raise "#{locale} isn't supported for release announce posts in blog." | |
end | |
__send__("post_#{locale}_content") |
post_content_#{locale}
にしてpost_content
をprefixにして関連処理であることをアピールしたほうがいいかも。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix: ed63a9e
ありがとうございます。わざわざ例外を挙げなくても、NoMethodErrorで気づけますし何をしているか、たしかにわかりやすいですね 🙏🏾
post_content_#{locale}にしてpost_contentをprefixにして関連処理であることをアピールしたほうがいいかも。
確かにです!
ここまで頂いたレビューコメントの対応を行いました! |
descriptionを最新情報に更新してもらえますか? |
すみません。更新しましたmm |
GitHub: GH-83
In this PR, we implement a Rake task for generating release announce posts with contents in blog.
Generate release announce posts