Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (245b296) 78.22% compared to head (7cdf4bc) 78.04%.

❗ Current head 7cdf4bc differs from pull request most recent head 8c4a9d6. Consider uploading reports for the commit 8c4a9d6 to get more accurate results

@@            Coverage Diff             @@
##             main    #2908      +/-   ##
==========================================
- Coverage   78.22%   78.04%   -0.18%     
==========================================
  Files         421      419       -2     
  Lines       42893    42793     -100     
==========================================
- Hits        33552    33398     -154     
- Misses       9341     9395      +54     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

redefines-builtin-id probably worth fixing, but it probably should be separate PR.

It'd be nice if we could put this in a golangci-lint yml, and use the golangci-lint github action, which will annotate the PR with code reviews https://github.com/golangci/golangci-lint-action

It'd be nice if we could put this in a golangci-lint yml, and use the golangci-lint github action, which will annotate the PR with code reviews https://github.com/golangci/golangci-lint-action

yeah, that's a good idea.

It'd be nice if we could put this in a golangci-lint yml, and use the golangci-lint github action, which will annotate the PR with code reviews https://github.com/golangci/golangci-lint-action

yeah, that's a good idea.

want me to merge this or do you want to do this as well?

It'd be nice if we could put this in a golangci-lint yml, and use the golangci-lint github action, which will annotate the PR with code reviews https://github.com/golangci/golangci-lint-action

yeah, that's a good idea.

want me to merge this or do you want to do this as well?

I think it better to merge this right now, and I will come with .conf file in other PR