Skip to content
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

Go 1.11 checking #155

Closed
jirfag opened this issue Jul 10, 2018 · 4 comments
Closed

Go 1.11 checking #155

jirfag opened this issue Jul 10, 2018 · 4 comments
Labels
enhancement New feature or improvement

Comments

@jirfag
Copy link
Member

jirfag commented Jul 10, 2018

No description provided.

@jirfag jirfag added the enhancement New feature or improvement label Jul 25, 2018
@wayneashleyberry
Copy link

wayneashleyberry commented Aug 14, 2018

I'm assuming this is a tracking issue for supporting go modules? I've been playing with 1.11-rc1 and managed to get everything to work except golangci-lint. I tried running go mod vendor after go mod download hoping that would help but to no avail.

WARN [runner/megacheck] Can't run megacheck because of compilation errors in packages [./pkg/mapper ./cmd/app ./cmd/cache ./cmd ./pkg/handler ./cmd/db ./pkg/cache ./pkg/app cloud.google.com/go/internal cloud.google.com/go/iam cloud.google.com/go/storage cloud.google.com/go/internal/trace]: pkg/mapper/mapper.go:7: could not import github.com/go-gorp/gorp (cannot find package "github.com/go-gorp/gorp" in any of:
	/usr/local/Cellar/go/1.10.3/libexec/src/github.com/go-gorp/gorp (from $GOROOT)
	/Users/wayneberry/go/src/github.com/go-gorp/gorp (from $GOPATH)) and 275 more errors: run `golangci-lint run --no-config --disable-all -E typecheck` to see all errors
pkg/mapper/mapper.go:7:2: could not import github.com/go-gorp/gorp (cannot find package "github.com/go-gorp/gorp" in any of:
	/usr/local/Cellar/go/1.10.3/libexec/src/github.com/go-gorp/gorp (from $GOROOT)
	/Users/wayneberry/go/src/github.com/go-gorp/gorp (from $GOPATH)) (typecheck)
	"github.com/go-gorp/gorp"
../../go/src/cloud.google.com/go/storage/writer.go:118:11: invalid operation: w.o.c.raw (variable of type *invalid type) has no field or method Objects (typecheck)
		call := w.o.c.raw.Objects.Insert(w.o.bucket, rawObj).
		        ^
../../go/src/cloud.google.com/go/storage/notifications.go:135:10: invalid operation: b.c.raw (variable of type *invalid type) has no field or method Notifications (typecheck)
	call := b.c.raw.Notifications.Insert(b.name, toRawNotification(n))
	        ^
../../go/src/cloud.google.com/go/storage/notifications.go:153:10: invalid operation: b.c.raw (variable of type *invalid type) has no field or method Notifications (typecheck)
	call := b.c.raw.Notifications.List(b.name)
	        ^
../../go/src/cloud.google.com/go/storage/notifications.go:166:28: invalid operation: res (variable of type *invalid type) has no field or method Items (typecheck)
	return notificationsToMap(res.Items), nil

That's just a sample of the errors, there are obviously a lot more.

@AlekSi
Copy link
Contributor

AlekSi commented Aug 25, 2018

Go 1.11 is released. Go issue referenced in .travis.yml was fixed.

@jirfag
Copy link
Member Author

jirfag commented Aug 28, 2018

now golangci-lint is built by go1.11, but go modules aren't supported yet

golangci pushed a commit that referenced this issue Aug 29, 2018
golangci pushed a commit that referenced this issue Aug 29, 2018
golangci pushed a commit that referenced this issue Aug 29, 2018
golangci pushed a commit that referenced this issue Aug 29, 2018
golangci pushed a commit that referenced this issue Aug 29, 2018
golangci pushed a commit that referenced this issue Aug 29, 2018
golangci pushed a commit that referenced this issue Aug 29, 2018
golangci pushed a commit that referenced this issue Aug 30, 2018
golangci pushed a commit that referenced this issue Aug 30, 2018
golangci pushed a commit that referenced this issue Aug 30, 2018
golangci pushed a commit that referenced this issue Aug 30, 2018
golangci pushed a commit that referenced this issue Aug 30, 2018
golangci pushed a commit that referenced this issue Aug 31, 2018
golangci pushed a commit that referenced this issue Aug 31, 2018
golangci pushed a commit that referenced this issue Aug 31, 2018
@jirfag
Copy link
Member Author

jirfag commented Sep 2, 2018

You can install golangci-lint via go modules but can't analyze go modules until we use go/packages

@jirfag jirfag closed this as completed Sep 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvement
Projects
None yet
Development

No branches or pull requests

3 participants