Skip to content

fix #296 - add go modules support#313

Open
s-kostyaev wants to merge 1 commit into
src-d:masterfrom
s-kostyaev:add-modules-support
Open

fix #296 - add go modules support#313
s-kostyaev wants to merge 1 commit into
src-d:masterfrom
s-kostyaev:add-modules-support

Conversation

@s-kostyaev

@s-kostyaev s-kostyaev commented Jul 5, 2019

Copy link
Copy Markdown

Add go modules support with golang.org/x/tools/go/packages

@s-kostyaev s-kostyaev force-pushed the add-modules-support branch from 4921266 to 780b45a Compare July 5, 2019 18:54
Signed-off-by: Sergey Kostyaev <sergey.kostyaev@bitfury.com>
@s-kostyaev s-kostyaev force-pushed the add-modules-support branch from 780b45a to 021d805 Compare July 5, 2019 18:56
@Julusian

Copy link
Copy Markdown

This works nicely for me.

I had to change the generate command to //go:generate go run gopkg.in/src-d/go-kallax.v1/generator/cli/kallax gen. It would be good to document this requirement in the readme.

And for anyone else wanting to test I added a temporary line to my go.mod file to use this PR: replace gopkg.in/src-d/go-kallax.v1 => github.com/s-kostyaev/go-kallax v1.3.6-0.20190705185553-021d805e3744

@iorlas

iorlas commented Jul 24, 2019

Copy link
Copy Markdown

Confirming, works good. It just needs to be confirmed to be working for non-mod projects for backward compat

@felipemfp

Copy link
Copy Markdown

@roobre, can you help to fix the failing tests?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants