Skip to content
This repository has been archived by the owner on Jun 28, 2018. It is now read-only.

Made source.Up and source.Down the same type #327

Closed
wants to merge 1 commit into from

Conversation

dcormier
Copy link

@dcormier dcormier commented Jan 3, 2018

Changed source.Up from being an untyped string const to being a typed source.Direction const. This makes source.Up and source.Down the same type.

Changed source.Up from being an untyped string const to being a typed source.Direction const.
@dcormier dcormier changed the title Changed source.Up from untyped to a typed const Made source.Up and source.Down the same type Jan 4, 2018
@dhui
Copy link

dhui commented Jan 20, 2018

Fixed in the fork: golang-migrate/migrate@3953915

@dcormier
Copy link
Author

dcormier commented Jan 26, 2018

Out of curiosity, why does that fork exist, @dhui? I don't see an explanation anywhere.

Ah, I see: #311.

@dhui
Copy link

dhui commented Jan 26, 2018

Sorry, I didn't link the issue explaining why I forked the repo. Thanks for linking it!

@dcormier
Copy link
Author

dcormier commented Mar 2, 2018

Closing as this is taken care of in the new maintained repo.

@dcormier dcormier closed this Mar 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants