Bh/pull out rails crap #17

Merged
merged 5 commits into from Jul 10, 2012

2 participants

@benhamill

Should delete all the rails stuff from the code. Eventually, we'll probably want to make a muddle-rails gem.

@benhamill

@kerinin Do you think we should get rid of the example directory, too? A rails app is a lot of code to be shipping around in a gem for purely documentation reasons.

@benhamill

This, BTW, starts work on #15.

@kerinin
OtherInbox member

Good point about the example rails app. That could be it's own repo too. I like having it around for integration testing.

You might push a reference branch before this gets merged so we can find all the removed files easily.

+1

@benhamill

I think if we want full-app integration testing stuff, we should write a little Sinatra thing. Or just have a thing that requires ActionMailer or whatever. Since we're moving the emphasis away from Rails integration.

@benhamill

Also, I created the tag rail_code_still_in. Awesome typo, huh? Oh well.

@benhamill

@kerinin 🆗 This is ready for review. Lemme know if you see anything weird.

@benhamill benhamill was assigned May 18, 2012
@kerinin
OtherInbox member

+1

@benhamill benhamill merged commit dc0b1d7 into master Jul 10, 2012
@benhamill benhamill referenced this pull request Jul 10, 2012
Closed

Pull out Rails crap #15

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