Skip to content

Update TargetRubyVersion to 3.2 (oldest maintained version)#111

Merged
tk0miya merged 2 commits intotimedia:masterfrom
epaew:feature/ruby/ci
Jun 9, 2025
Merged

Update TargetRubyVersion to 3.2 (oldest maintained version)#111
tk0miya merged 2 commits intotimedia:masterfrom
epaew:feature/ruby/ci

Conversation

@epaew
Copy link
Member

@epaew epaew commented May 30, 2025

関連する Issues

やったこと

.github/workflows, ruby/.rubocop.yml の対象Rubyバージョンを3.2に変更する

@epaew
Copy link
Member Author

epaew commented May 30, 2025

rubocop/rubocop-config-timedia.gemspec:7:32: W: Gemspec/RequiredRubyVersion: required_ruby_version and TargetRubyVersion (3.2, which may be specified in .rubocop.yml) should be equal.
  spec.required_ruby_version = '>= 2.7.0'
                               ^^^^^^^^^^

なるほど、しかしここを変えちゃっていいのかは利用者一通りに聞かないとわかんないな…

逆に「required_ruby_version を消す」選択肢もアリかな?

@epaew epaew force-pushed the feature/ruby/ci branch from 246aa26 to a22b8c7 Compare May 30, 2025 10:25
@tk0miya
Copy link
Member

tk0miya commented May 31, 2025

slack に流して、リアクションがなければあげちゃっていいと思います。

一部の例外を除くと、もう使ってないのでは、と思いたい…
(そして、その一部の例外は独自のルールを適用しているので問題ないはず)

@epaew epaew force-pushed the feature/ruby/ci branch from a22b8c7 to b749acd Compare June 9, 2025 07:56
@github-actions github-actions bot added the ruby label Jun 9, 2025
@epaew
Copy link
Member Author

epaew commented Jun 9, 2025

rubocop/rubocop-config-timedia.gemspec:7:32: W: Gemspec/RequiredRubyVersion: required_ruby_version and TargetRubyVersion (3.2, which may be specified in .rubocop.yml) should be equal.
  spec.required_ruby_version = '>= 2.7.0'
                               ^^^^^^^^^^

なるほど、しかしここを変えちゃっていいのかは利用者一通りに聞かないとわかんないな…

逆に「required_ruby_version を消す」選択肢もアリかな?

消すことにした。

「結局 RuboCop の gemspec に従う」だけなので、であれば

  • 消しても問題になることはない
  • 残しておくことで「追従しなきゃいけない」手間が増える

と考えた

@epaew epaew marked this pull request as ready for review June 9, 2025 08:00
@epaew epaew requested review from a team as code owners June 9, 2025 08:00
Copy link
Member

@tk0miya tk0miya left a comment

Choose a reason for hiding this comment

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

gemspec については特に意見がないので +1 しました。
(書いてあってもいいし、消しても構わない)

@tk0miya tk0miya merged commit 7dea022 into timedia:master Jun 9, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants