Yeah, if we want to follow semver strictly, we would need to. On the other
hand, we could argue that this class is not part of the public interface
(in contrast to the methods in the `Lhm` module or the methods in
`Migrator` class).


On Fri, Jul 5, 2013 at 10:40 PM, Durran Jordan <notifications@github.com>wrote:

> In lib/lhm/migration.rb:
>
> >
> > -    def initialize(origin, destination, time = Time.now)
> > +    def initialize(origin, destination, conditions = nil, time = Time.now)
>
> Bump as major release?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/soundcloud/lhm/pull/44/files#r5045355>
> .
>