You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The reason will be displayed to describe this comment to others. Learn more.
There are currently no breaking changes. Why are we diverging development into multiple branches? Isn't it enough to just list supported versions in the readme and gemspec?
The reason will be displayed to describe this comment to others. Learn more.
Ok, so I missed a couple of changes, however, I see no reason we couldn't maintain support for both AR versions in 1 gem. I'll put together a couple of pull requests, and we can discuss further.
2611574
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are currently no breaking changes. Why are we diverging development into multiple branches? Isn't it enough to just list supported versions in the readme and gemspec?
2611574
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, so I missed a couple of changes, however, I see no reason we couldn't maintain support for both AR versions in 1 gem. I'll put together a couple of pull requests, and we can discuss further.
2611574
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There appear to be some changes to migrations between Rails 3 and Rails 4 (see #41 and #45). Will you be able to reconcile these changes in #47?
2611574
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My thinking was to make the version number:
delayed_job
that it depends onrails
that it supports (this is more of a happy coincidence)