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

release: add support the format of Mroonga version #93

Merged
merged 2 commits into from
Dec 26, 2024

Conversation

komainu8
Copy link
Member

Mroonga version format is "XX.YY".
However, /\d+\.\d+\.\d+/ does not match "XX.YY".

Copy link
Contributor

@otegami otegami left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍🏾

@@ -146,8 +146,8 @@ def define_version_update_task
latest_release = GitHubClient.new(@product, @product).latest_release
# "Groonga 14.1.1 - 2024-12-03"
release_name = latest_release["name"]
# "14.1.1"
latest_version = release_name[/\d+\.\d+\.\d+/, 0]
# "14.1.1" or "14.11"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

上の "Groonga 14.1.1 - 2024-12-03" と連動しているコメントだと思うので、上の方にも or "Mroonga 14.11 - 2024-12-03" とか書いておくとわかりやすそうです。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

追記しました!

@abetomo abetomo merged commit 6b27908 into gh-pages Dec 26, 2024
1 check passed
@abetomo abetomo deleted the support-mroonga-version-string branch December 26, 2024 05:38
@kou
Copy link
Member

kou commented Dec 29, 2024

次からは自分のforkから出して。

@komainu8
Copy link
Member Author

komainu8 commented Jan 6, 2025

次からは自分のforkから出して。

承知しました。

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

Successfully merging this pull request may close these issues.

4 participants