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

Enable gocritic in missing components #10466

Closed
70 tasks done
Tracked by #9109
mx-psi opened this issue May 31, 2022 · 8 comments
Closed
70 tasks done
Tracked by #9109

Enable gocritic in missing components #10466

mx-psi opened this issue May 31, 2022 · 8 comments
Assignees
Labels

Comments

@mx-psi
Copy link
Member

mx-psi commented May 31, 2022


List generated by running:

❯ git diff-tree --no-commit-id --name-only -r 0c691295193854523240b609c0c3cc06f9202b41 | rg .go$ | xargs dirname | sort | uniq | awk '{print "- [ ] Enable gocritic in " $0}'
@fatsheep9146
Copy link
Contributor

@mx-psi please assign this to me :)

@TylerHelmuth TylerHelmuth added the priority:p2 Medium label Jun 9, 2022
@fatsheep9146
Copy link
Contributor

fatsheep9146 commented Aug 6, 2022

@mx-psi I have checked all component, for gocritic only two components are still not resolved, others can all be checked

@mx-psi
Copy link
Member Author

mx-psi commented Aug 9, 2022

Nice! Looks like pkg/telemetryquerylanguage/contexts/tqlmetrics/metrics.go is also missing, otherwise we are almost finished :)

@evan-bradley
Copy link
Contributor

@mx-psi Can this issue be closed?

@mx-psi
Copy link
Member Author

mx-psi commented Sep 30, 2022

❯ rg -tgo gocritic -l | cut -d '/' -f 1-2 | sort | uniq
pkg/ottl
receiver/awscontainerinsightreceive

These are the two components missing:

@TylerHelmuth
Copy link
Member

@mx-psi I'll take a look

@fatsheep9146
Copy link
Contributor

❯ rg -tgo gocritic -l | cut -d '/' -f 1-2 | sort | uniq
pkg/ottl
receiver/awscontainerinsightreceive

These are the two components missing:

I think this issue is ready to be closed. =D @mx-psi

@mx-psi
Copy link
Member Author

mx-psi commented Oct 13, 2022

Indeed 🎉 🎉 Thanks for your hard work on this @fatsheep9146 !

@mx-psi mx-psi closed this as completed Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants