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

apply rubocop 1.59.0. Drop Ruby 2.7 and add 3.3 #1905

Merged
merged 3 commits into from
Jan 6, 2024
Merged

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Jan 6, 2024

Rubocop 1.59.0を適用しました。
bundlerがもうだめっぽいので2.7をやめて、3.3を追加しました。

@kmuto
Copy link
Owner Author

kmuto commented Jan 6, 2024

jlreqがb5→b5j化したのはTeXLive 2021のため、Ubuntuイメージを更新しました。
両立させるのはかなり難しいのとあまりがんばる価値がないのであきらめ。

@kmuto kmuto merged commit 81e9b45 into master Jan 6, 2024
17 checks passed
@kmuto kmuto deleted the rubocop1_59_0 branch January 6, 2024 04:45
@takahashim
Copy link
Collaborator

このPRとは直接関係ないですが、CIはさておき実際のサポートするRubyのバージョンはいくつまでにしますか?
たとえば ReVIEW::YAMLLoader のRuby 2.4以下の対応用コードはいつまで残しておくかみたいな話です。

@kmuto
Copy link
Owner Author

kmuto commented Jan 7, 2024

@takahashim
気持ちとしては、どうしてもな理由がなければDebian oldstableに収録されているバージョンまではサポートしたいと思っています。

https://wiki.debian.org/DebianReleases
Bullseye EOL 2024-07

https://packages.debian.org/search?keywords=ruby
bullseye (oldstable) (ruby): オブジェクト指向スクリプト言語 Ruby のインタプリタ (デフォルト版)
1:2.7+2: amd64 arm64 armel armhf i386 mips64el mipsel ppc64el s390x

ということで、Ruby 2.7はサポートしたい、2.4は捨ててもいい ですね。
(:"〜"ってもしかしてダメだったりする?)

@takahashim
Copy link
Collaborator

ということで、Ruby 2.7はサポートしたい、2.4は捨ててもいい ですね。

なるほど、これは了解です。

(:"〜"ってもしかしてダメだったりする?)

これはどういう意味ですか? (Gemfileやgemspecでの~>の話でしょうか)

@kmuto
Copy link
Owner Author

kmuto commented Jan 10, 2024

舌足らずですみません、 https://github.com/kmuto/review/pull/1905/files#diff-dcab263f4de04d58aa6f14224b6f924b205d0787ec181b1b594818c92ee33fa4R643
の書き方はrubocop指摘でこうしたのですが、2.7で大丈夫だったかなと不安になったのでした。
Docker環境で試してみましたが大丈夫そうですね。

@takahashim
Copy link
Collaborator

あーなるほど、シンボルのリテラルの話だったんですね。これはだいぶ前からサポートされてたはずです(調べたら2.0.0のマニュアルにも記載がありました)。
https://docs.ruby-lang.org/ja/2.0.0/doc/spec=2fliteral.html#symbol

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.

2 participants