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

Fix go generate #126

Merged
merged 1 commit into from
Oct 11, 2021
Merged

Conversation

yitsushi
Copy link
Contributor

What this PR does / why we need it:

Fix go generate

❯ make generate-go
go generate ./...
/home/efertone/go/pkg/mod/github.com/google/[email protected]/internal/wire/copyast.go:21:2: missing go.sum entry for module providing package golang.org/x/tools/go/ast/astutil (imported by github.
com/google/wire/internal/wire); to add:
        go get github.com/google/wire/internal/[email protected]
/home/efertone/go/pkg/mod/github.com/google/[email protected]/internal/wire/parse.go:30:2: missing go.sum entry for module providing package golang.org/x/tools/go/packages (imported by github.com/g
oogle/wire/internal/wire); to add:
        go get github.com/google/wire/internal/[email protected]
/home/efertone/go/pkg/mod/github.com/google/[email protected]/internal/wire/analyze.go:26:2: missing go.sum entry for module providing package golang.org/x/tools/go/types/typeutil (imported by gith
ub.com/google/wire/cmd/wire); to add:
        go get github.com/google/wire/cmd/[email protected]
internal/inject/wire_gen.go:3: running "go": exit status 1
make: *** [Makefile:73: generate-go] Error 1

Which issue(s) this PR fixes:
Fixes #125

Checklist:

  • squashed commits into logical changes

@yitsushi yitsushi added kind/cleanup Removing things previously overlooked kind/bug Something isn't working and removed kind/cleanup Removing things previously overlooked labels Oct 11, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #126 (a3d991e) into main (12b503c) will increase coverage by 0.13%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #126      +/-   ##
==========================================
+ Coverage   40.09%   40.23%   +0.13%     
==========================================
  Files          32       32              
  Lines        1444     1444              
==========================================
+ Hits          579      581       +2     
+ Misses        789      788       -1     
+ Partials       76       75       -1     
Impacted Files Coverage Δ
pkg/queue/queue.go 100.00% <0.00%> (+6.06%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6578913...a3d991e. Read the comment docs.

@yitsushi yitsushi merged commit a284a85 into liquidmetal-dev:main Oct 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Go generate fails
3 participants