-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use the current path to go-migrate #143
Conversation
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.
LGTM, but left a question
fs-repo-11-to-12/go.mod
Outdated
@@ -4,12 +4,11 @@ go 1.15 | |||
|
|||
require ( | |||
github.com/hsanjuan/ipfs-lite v1.1.19 | |||
github.com/ipfs/fs-repo-migrations v1.7.1 | |||
github.com/ipfs/fs-repo-migrations/tools v0.0.0-20210323144402-297a63449538 |
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.
Any reason we're not using a released version here?
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.
I'm not even sure this is in master, might just be on your local machine from when we did this last time
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.
I cannot use a version tag, as the module is not in the root of the repo.
go get: module github.com/ipfs/[email protected] found (v2.0.2+incompatible), but does not contain package github.com/ipfs/fs-repo-migrations/tools
If I update to the commit of v2.0.2, that gives me:
github.com/ipfs/fs-repo-migrations/tools v0.0.0-20210617204053-ea2bc1a62948
So, we can use that if you prefer.
This has failed for something stupid, I'm merging and rebasing #145 on top, and we take those tests as proper signal. |
No description provided.