I feel like this would be better suited inside this ```is_package_shippable``` https://github.com/spree/spree_active_shipping/blob/master/app/models/spree/calculator/shipping/active_shipping/base.rb#L77-L82 i made that to check if this package is able to get rates, would you agree?