-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
Rubyのバージョンを3.2.8にする #1684
Conversation
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.
気になっているところをコメント
app/models/podcast.rb
Outdated
@@ -21,7 +21,7 @@ def path | |||
def exists?(offset: 0) |
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.
【気になっている】 今回は標準ライブラリの File#exist?
だけを変更していますが、このメソッドもそちらに合わせた方がよいか迷っています。
Document#exists?
なども定義しているので、Podcast#exists?
もそのままでよいかな?と判断したのですがフィードバックいただけると幸いです 🙏
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.
@kimihito PR ありがとうございます!😻🆒✨
ダックタイピングしているものなので他のも合わせて exist?
に変えてもらえると助かります!(>人< )✨
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.
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.
早速のご対応ありがとうございます!マージしますね...!! 😻🆒✨
61b286b
to
e8dcdf5
Compare
対応Issue
#1683 の対応です
やったこと
表題の通り、Rubyのバージョンを3.1.6から3.2.8に変更しました。
close #1683