burke / zeus

Boot any rails app in under a second.
MIT License
3.33k stars 231 forks source link

Revert #530 (requires using Zeus with bundler) #573

Closed metcalf closed 8 years ago

metcalf commented 8 years ago

This reverts #530.

The README currently advises running Zeus outside of bundler. I agree that running Zeus without bundler is probably a bad tradeoff in terms of stability but I don't think we can change this without breaking everyone's Zeus integrations. I believe we should revert until we're prepared to re-introduce this with changes to the README, better error messaging to help people transition and possibly a major version bump.

metcalf commented 8 years ago

review? @latortuga @kgrz

metcalf commented 8 years ago

ping @latortuga @kgrz! I'd love to get at least a bit of feedback on this and it's blocking cutting a new release.

kgrz commented 8 years ago

LGTM 👍 other than those two minor comments