whiteleaf7 / narou

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

add development dependencies to narou.gemspec #348

Closed topstone closed 4 years ago

topstone commented 4 years ago

それぞれの開発時用 gems の版は、よく分からないので指定していません。(この pull request を merge していただいた後に) 必要に応じて指定していただければと思いますが、とりあえず現状では版指定しなくても (現時点の最新版でも) 動作しているように見えます。

whiteleaf7 commented 4 years ago

数年前のことで記憶があやふやですが、たしか add_development_dependency を指定してないのは意図的です 開発用のライブラリは Gemfile に記述しているので、開発用の環境を整えるのに bundle install を使うのを想定しています

理由として、Narou.rbは単体で完結しているアプリでありライブラリではないので、アプリ自体には開発用の依存関係は必要ありません 開発用としては bundler を使ったほうが便利だから、、、だったかな?

whiteleaf7 commented 4 years ago

Gemfile を見ればわかりますけど、開発用だけで10ライブラリあるので、これを全部 gemfile に書きたくないって理由もあった気がします https://rubygems.org/gems/narou に依存ライブラリとして全部載っちゃうんですよね確か Gemfile に書けば出てこないので…

topstone commented 4 years ago

回答ありがとうございます。

gem 内で固定的なものが Gemfile に書いてあるとは想定しておりませんでした。(通常の gem であれば、gem install narou --development のように入力するだけで開発に必要なものが揃うので、わざわざ Bundler を使う人はあまりいないかと思います。)

Qiita の「gemライブラリの依存はGemfileではなくgemspecに記述する理由」という記事でも「Gemfileはgemライブラリの依存を定義するものとしては使えません!!」と書いてありますし、私も昔から同意見なので、できれば Gemfile ではなく narou.gemspec に書いていただく方が大変ありがたいのではありますが…

「開発用だけで10ライブラリあるので、これを全部 gemfile に書きたくないって理由もあった気がします」というのであれば、強くお願いすることはできないですね… (私の感覚だと、たった10個なのに、と思ってしまいます。)

まあ、私の不満は私が fork すればよいことなので、戯言だと思って無視して下さい。

大変有用な gem を公開していただき、本当にありがとうございます。

whiteleaf7 commented 4 years ago

どうしてこんな書き方をしてるかってのを思い出してるだけなので、 提示していただいた資料を見る限り gemspec に書いたほうが良さそうですね 最初に作ったの2012年?ごろなので正直細かいこと考えてませんでしたね

whiteleaf7 commented 4 years ago

(私の感覚だと、たった10個なのに、と思ってしまいます。)

これですけど、私の今の感覚だと同じでたった10個、ですねw まぁアプリ開発には数十個使うのは当たり前ですけど、でも rubygems の公開ページにずらずら並ぶのはまた別のインパクトがあります

topstone commented 4 years ago

ありがとうございます。gemspec に書いていただければ大変助かります。

まぁアプリ開発には数十個使うのは当たり前ですけど、でも rubygems の公開ページにずらずら並ぶのはまた別のインパクトがあります

私は数が多くても全く気にならない人ですが (依存 gems がどんなに多くても機械的に処理されるだけなので)、もし気になるようでしたら、narou-core, narou-cli, narou-webserver, … のように機能ごとに gem を分割するとよいかも知れません (分割時にそこそこ手間がかかるので、 @whiteleaf7 さんの判断次第ですが…)。

分割されれば、私のように CUI しか使わない人はそこだけ RSpec を通せばいいですし、web server に不満がある人は narou-core を直接叩く gem を自作すればよいので、利点もそれなりにあるかなと思います。

whiteleaf7 commented 4 years ago

こちらはきちんと対応する予定なのでお待ち下さい

whiteleaf7 commented 4 years ago

書き直したので一旦閉じ