It doesn't actually depend on rails, just railties

[rosenfeld]

This allows an application not to download the activerecord and arel gems for instance, if they don't use such ORM. As long as the application itself doesn't depend on the rails gem but railties instead.

I've actually tested this locally in my own application since I use Sequel as the ORM solution.

The only gem I depend on that still depends on rails is js-routes. Please consider applying this patch.

[le0pard]

Hello, @rosenfeld . I think, you should add new test env to https://github.com/railsware/js-routes/blob/master/Appraisals, generate new gemspec (rake appraisal:gemfiles and rake appraisal:install) which not contain Rails and check what all tests pass without it.

[rosenfeld]

Ok, I'll try to give it a try later today.

[rosenfeld]

Indeed, I missed another required js-routes dependency, added by the rails gem: sprockets-rails :)

[bogdan]

js-routes definitely depends on actionpack. Please include it as well if it is not part of railties dependencies.

[rosenfeld]

Railties already depends on actionpack: https://github.com/rails/rails/blob/master/railties/railties.gemspec#L27

[le0pard]

@rosenfeld Why we need tzinfo gem in tests?

[rosenfeld]

The Rails application itself seems to need it probably due to some default configs, but this is not a js-routes dependency. All existing Rails applications will either have it or won't need tzinfo. I don't think we should worry about it.

[rosenfeld]

Otherwise we should change the test application defaults not to rely on tzinfo, but I think this may be harder then we may initially think it would... And more fragile too...

[le0pard]

@rosenfeld As I can see activesupport from 4.x.x have in dependency tzinfo, and railties have in dependency activesupport. So we can remove it from section rails40. And maybe it no need for railties 3.x.x

[rosenfeld]

I added it because some test failed without it, but maybe you're right about rails40, didn't test it without that dependency. Will do and let you know.

[rosenfeld]

That's right, rails40 doesn't need it. I'll remove it and push the changes

[rosenfeld]

done

[le0pard]

Looks good. Need wait for @bogdan - what he will say.

[bogdan]

Also looks good for me. Merging in.

[rosenfeld]

Thank you :) Any estimates for next release? ;)

[le0pard]

+1 for release, because master have fix for IE :)

[bogdan]

check 0.9.7

[rosenfeld]

Thanks :-)