spine / spine-rails

Spine plugin for Rails
171 stars 38 forks source link

Update to v1.10 #36

Closed whatyoubendoing closed 11 years ago

whatyoubendoing commented 11 years ago
zohararad commented 11 years ago

@benhainez how shall we proceed with this. How would you like to divide work on the Gem?

whatyoubendoing commented 11 years ago

@zohararad ah not sure, have you had a check over the code in this pull reques? If you are happy for me to merge this back in we can go from here.

@maccman how would you like to go about managing the deployment of this gem?

zohararad commented 11 years ago

@benhainez I went over the commits, and they seem fine to me and very much inline with keeping this gem up to date.

My only very slight concern is adding some sort of tests to make sure everything is there.

I can try and add these after you merge and before you release by week's end, unless you have time to do it before but i dont think it's a show stopper.

whatyoubendoing commented 11 years ago

Yes there is one bug with Spine 1.1.0 which will be included in this pull request. However I am going to merge the request tag it and import the latest from master. One other thing we need to update is the generators as jQuery.sub() is depreciated in jQuery version 1.9