'coveralls' missing as a build dependency in gemspec about wisper HOT 7 CLOSED

Coveralls is included in the Gemfile, which allows it to be built locally.

I'm curious, why are you trying to build a Debian package with the test suite included?

from wisper.

@bmorrall the debian infrastructure runs test suites automatically when building packages whenever they are present. we like to configure them so they run properly: this way we can more easily see if a change in dependency libraries or ruby version has the effect of breaking functionality in some package.

from wisper.

🤔 Is it possible to do bundle install as part of the build step for the tests this will install all the development/test dependencies which are in Gemfile?

from wisper.

There is a second option and that is something like:

begin
  require 'coveralls'
rescue LoadError
  # no-op
else
  Coveralls.wear!
end

from wisper.

@krisleech so I sought out some feedback from fellow ruby packagers since I'm still very fresh in that set of tech skills. I should've known better, but ain't that always the case for something -- especially for beginners hehe. coveralls is not required at all for the purpose of running the test suite, so I could've patched it out of spec_helper.rb.

the patch you suggested in your last message seems to answer perfectly to that need, so if you and other contributors are comfortable merging that in, it would solve the issue for me. however it might mean that some developers won't update coverage information systematically since it won't be compulsory to run coveralls anymore. if some ppl are not super comfortable, I can patch it at packaging level.

from wisper.

@lelutin Personally, I'm using env vars to be able to "opt-out" in such situations if packaging stuff for Debian.

from wisper.

I'm happy with the rescue no-op solution if someone wants to submit a PR.

from wisper.