Skip to content

Conversation

@rubeniskov
Copy link

No description provided.

@rubeniskov rubeniskov changed the base branch from master to develop June 3, 2020 13:35
@Kelvur
Copy link
Member

Kelvur commented Jun 6, 2020

I'm getting this error
image

I can't resolve the dependencies

I try the same command in other repositories and just works fine

This can be because reposiroty_test.go is importing "github.com/lot-sh/core/api" ?

@Kelvur
Copy link
Member

Kelvur commented Jun 6, 2020

I just realice the branch this branch is diferent then feature/implement_repository_filesystem, the i is in lowercase

@Kelvur
Copy link
Member

Kelvur commented Jun 6, 2020

I just realice the branch this branch is diferent then feature/implement_repository_filesystem, the i is in lowercase

But I get the same error

@rubeniskov
Copy link
Author

rubeniskov commented Jun 6, 2020

This error is related to the use of -u flag, try to execute the same command without that. I understand why are you interest to solve the dependencies forcing downloading instead of getting the current version of your local packages, but to keep the consistency of the dependencies and their version we will use go modules, but at this point we don't need it.

golang/go#34117

@Kelvur
Copy link
Member

Kelvur commented Jun 7, 2020

The thing is I don't have local packages before trying go get -v -u -f, the command go get just don't work and also doesn't print a error. The problem now is gone, in my research I found the command go mod init, that resolve the problem, there is a possibility you have a file go.mod put don't add it to git?

@Kelvur
Copy link
Member

Kelvur commented Jun 18, 2020

@rubeniskov added style, if you think the style is correct this branch can be merged

@Kelvur Kelvur added the enhancement New feature or request label Jun 18, 2020
@Kelvur
Copy link
Member

Kelvur commented Jun 18, 2020

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants