whiteleaf7 / narou

Narou.rb - 小説家になろうのダウンローダ&縦書き整形&管理アプリ。Kindle(などの電子書籍端末)でなろうを読む場合に超便利です!
Other
496 stars 55 forks source link

URI.regexp 書き直し #355

Closed topstone closed 4 years ago

topstone commented 4 years ago

Ruby 2.2 の頃より URI.regexp は obsolete のようです。一応現状でも動きますが、いずれ単なる obsolete から error になるのではないかと思います。

とりあえず downloader.rb だけ書き直し、RSpec が all green になることを確認しました。他にも以下の2か所に残っています (手を付けていません)。

topstone commented 4 years ago

すみません、rubocop 通っていませんでした。一旦 close して、また後日 PR 出します。

topstone commented 4 years ago

現状報告です。

  1. https://narou.nyanpass.jp/ 閉鎖
  2. narou.rb 発見
  3. Ruby 2.7 で使えない (常に最新の Ruby を使いたい派なので…)
  4. PRs 提出 (自分の手元では Ruby 2.7 でも動くようになった)
  5. Ruby 2.8 以降へも移行しやすいよう、ruby -w で warning 検出
  6. まずは URI.regexp から挑戦
  7. rubocop で引っかかる
  8. rubocop --safe-auto-correct で URI.regexp も自動修正されるっぽい
  9. RSpec で error 発生 (safe 修正のはずなのに…)
  10. rubocop の bug っぽい
  11. rubocop へ bug report ←今ココ
whiteleaf7 commented 4 years ago

rubocop --safe-auto-correct で URI.regexp も自動修正されるっぽい

ほお〜

whiteleaf7 commented 4 years ago

Ruby 2.7では動かないと注釈しているにも関わらず、2.7 で動かないと不具合報告してくる人が現れてきてしまったので、 そろそろPR頂いたやつを反映します

topstone commented 4 years ago

Ruby 2.7では動かないと注釈しているにも関わらず、2.7 で動かないと不具合報告してくる人が現れてきてしまったので、

とりあえず、narou.gemspec の中で gem.required_ruby_version = ">=2.3.0", "<2.7.0" として narou 3.4.9 などという形で release してはいかがでしょうか。注釈に書くよりも gemspec に書く方が確実かと思います。

私の手元で勝手に (上記の形の) narou-3.4.9.gem を build して Ruby 2.7 で導入しようとしたら、ちゃんと There are no versions of narou (= 3.4.9) compatible with your Ruby & RubyGems narou requires Ruby version >= 2.3.0, < 2.7.0. The current ruby version is 2.7.0.0. と表示されました。

…と私が書いているうちに、narou 3.5.0 化が進んでいましたね。失礼しました。

whiteleaf7 commented 4 years ago

私の手元で勝手に (上記の形の) narou-3.4.9.gem を build して Ruby 2.7 で導入しようとしたら、ちゃんと There are no versions of narou (= 3.4.9) compatible with your Ruby & RubyGems narou requires Ruby version >= 2.3.0, < 2.7.0. The current ruby version is 2.7.0.0. と表示されました。

というエラーがでてしまったのですが、という報告がくるだけに一票 結局サポートコストは変わらないのだった、完

narou 3.5.0 を gem push したのでご確認ください

topstone commented 4 years ago

narou 3.5.0 ありがとうございます。

rubocop に出した bug report は、patch が作られて master に取り込まれましたが、rubocop 0.80.0 には間に合わなかったようです。

次回 rubocop が release されたら、よければ rubocop --safe-auto-correct を試して RSpec を回してみてください。

topstone commented 4 years ago

続き : #360