ckdake / setler

Setler lets you use the 'Feature Flags' pattern or add settings to models in Rails
MIT License
84 stars 21 forks source link

Add non-TravisCI changes to support multiple Rails versions #35

Closed jasnow closed 9 years ago

jasnow commented 9 years ago

Add non-TravisCI changes to support multiple Rails versions (issues #33)

ckdake commented 9 years ago

Is there a reason to have this README as a separate file or could we go ahead and make this change in the main one?

jasnow commented 9 years ago

Is there a reason to have this README as a separate file or could we go ahead and make this change in the main one?

This feature is not done (needs TravisCI part) so I did not want to change the external project information until it was done.

ckdake commented 9 years ago

Gotcha, are there instructions for what I need to do in Travis to get this all playing nice?

jasnow commented 9 years ago

Gotcha, are there instructions for what I need to do in Travis to get this all playing nice?

The large number of my previous commits were generally focused on trying to solve the Travis integration. I have not solved that integration so I decided to create this pull request which includes all but the Travis integration. The Travis integration would come in a future pull request.

If you do not want this partial solution, just add a comment to this PR and I will close it. Cheers....

ckdake commented 9 years ago

Lets done one of:

  1. Split this into two pieces then, one for the partial solution and one for the Travis bit (with the new readme in the Travis PR only)
  2. Add a "Coming Soon" part to the README with the Travis specific things that aren't ready yet
jasnow commented 9 years ago

I will adding the (2) "Coming Soon" section to the new README file. Then support above (1) with this pull request

jasnow commented 9 years ago

Next we can replace original README with new version of README if you like the changes. To see new version, use this: https://github.com/jasnow/setler/blob/master/README-appraisal.rdoc

ckdake commented 9 years ago

Looks good, If you remove the README-appraisal file from this PR, I'll merge this on in.

jasnow commented 9 years ago

Removing README-appraisal file as requested.

jasnow commented 9 years ago

THANKS !!!!