-
Notifications
You must be signed in to change notification settings - Fork 27
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
deps: Use internal/tools
for tools rather than install from Makefile
#1698
Conversation
2cd43b2
to
67b65e1
Compare
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.
CI fails with
Error: main.go:19:2: import "github.com/client9/misspell/cmd/misspell" is a program, not an importable package
It looks right at a glance. I am not sure what the deal is.
Yea, not entirely sure what the issue is with it either. BPOP has the same issue/error and we resolve it by adding it to an ignore list for gosec, so I did that here and it should be passing now. |
I think the |
cfd01d5
to
c88073a
Compare
c88073a
to
12e1999
Compare
Proposed Change
Updating OTel to v0.103.0 introduces a breaking change in code that mdatagen generates. We've put off updating mdatagen for a while now because doing so caused issues but now we are forced to update it to pull in the new OTel update. In order to update mdatagen (will be done in PR after this one) we need to change how we handle tools.
Checklist