General modernization #71

Merged
merged 5 commits into from Feb 3, 2013

Conversation

Projects
None yet
3 participants
Contributor

teeparham commented Jan 23, 2013

Hello! This is a major PR that hopefully cleans up a lot of things.

Tests pass & I built a local gem that works in our rails app.

Thanks!

Owner

diogob commented Jan 24, 2013

Wow, nice patch :)
I'll take a look in to it and merge it ASAP

Contributor

joaomilho commented Jan 24, 2013

awsm

Contributor

teeparham commented Jan 24, 2013

Thanks. I'm not sure I have the platform-dependent gems defined exactly right. Also we may want to specify a minimum rails version, like '>= 3.0'.

One thing I forgot to note: run rake -T to see the gem build tasks. You now can do rake build and rake release to build & release the gem. Run rake install to build & install the gem locally. Then do bundle install --local to test the local version.

@diogob diogob merged commit 70ae4b0 into diogob:master Feb 3, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment