Skip to content

Rubyのバージョンを3.2.8にする #1684

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

Merged
merged 4 commits into from
May 17, 2025

Conversation

kimihito
Copy link
Member

対応Issue

#1683 の対応です

やったこと

表題の通り、Rubyのバージョンを3.1.6から3.2.8に変更しました。

close #1683

@kimihito kimihito changed the title Bump Ruby version to 3.2.8 Rubyのバージョンを3.2.8にする May 16, 2025
@kimihito kimihito marked this pull request as draft May 16, 2025 13:53
Copy link
Member Author

@kimihito kimihito left a comment

Choose a reason for hiding this comment

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

気になっているところをコメント

@@ -21,7 +21,7 @@ def path
def exists?(offset: 0)
Copy link
Member Author

Choose a reason for hiding this comment

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

【気になっている】 今回は標準ライブラリの File#exist? だけを変更していますが、このメソッドもそちらに合わせた方がよいか迷っています。

Document#exists? なども定義しているので、Podcast#exists? もそのままでよいかな?と判断したのですがフィードバックいただけると幸いです 🙏

Copy link
Member

@yasulab yasulab May 17, 2025

Choose a reason for hiding this comment

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

@kimihito PR ありがとうございます!😻🆒✨

ダックタイピングしているものなので他のも合わせて exist? に変えてもらえると助かります!(>人< )✨

Copy link
Member Author

Choose a reason for hiding this comment

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

@yasulab e8dcdf5 で対応してみました!

Copy link
Member

Choose a reason for hiding this comment

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

早速のご対応ありがとうございます!マージしますね...!! 😻🆒✨

@kimihito kimihito marked this pull request as ready for review May 16, 2025 14:12
@kimihito kimihito force-pushed the chore/bump-ruby-3.2.8 branch from 61b286b to e8dcdf5 Compare May 17, 2025 05:58
@yasulab yasulab merged commit a382534 into coderdojo-japan:main May 17, 2025
2 checks passed
@yasulab yasulab mentioned this pull request May 20, 2025
8 tasks
@kimihito kimihito deleted the chore/bump-ruby-3.2.8 branch May 21, 2025 08:22
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.

rubyのバージョンを3.2にアップグレードする
2 participants