Bullet should not automatically require ActiveRecord / Mongoid

[johnnyshields]

Currently Bullet forces these libraries to be required in your application, however, this is not a good practice since many apps many want to intentionally exclude one of these libraries (like mine). Instead, it is better to check if they are already defined (which would have been done earlier in the Rails initialization chain)

[coveralls]

Coverage decreased (-0%) when pulling dd105d9 on johnnyshields:master into 0ae36b9 on flyerhzm:master.

[johnnyshields]

FYI I've tested this new version in my app (with Mongoid) and it looks like it's working normally

[flyerhzm]

it breaks travis ci, I think you should rescue LoadError, like

begin
  require 'mongoid'
rescue LoadError
end

[coveralls]

Coverage remained the same when pulling 5211da6 on johnnyshields:master into 0ae36b9 on flyerhzm:master.

[coveralls]

Coverage decreased (-0%) when pulling b50a664 on johnnyshields:master into 0ae36b9 on flyerhzm:master.

[flyerhzm]

@johnnyshields I still see some errors on travis ci, could you run sh test.sh on your local machine?

[johnnyshields]

Should be good now

[flyerhzm]

nice