Skip to content

Conversation

@akhileshsingh85
Copy link
Contributor

@akhileshsingh85 akhileshsingh85 commented Nov 25, 2025

Description: Enable all the linter and resolve the issue

Testing: Golint test

Documentation:

@codecov
Copy link

codecov bot commented Nov 25, 2025

Codecov Report

❌ Patch coverage is 47.05882% with 81 lines in your changes missing coverage. Please review.
✅ Project coverage is 33.63%. Comparing base (0ede663) to head (01d7d8f).

Files with missing lines Patch % Lines
tests/testutils/collector_container.go 21.05% 15 Missing ⚠️
tests/testutils/collector_process.go 27.27% 8 Missing ⚠️
tests/testutils/testcase.go 0.00% 7 Missing ⚠️
internal/settings/settings.go 54.54% 4 Missing and 1 partial ⚠️
tests/testutils/expvarz_to_map.go 0.00% 5 Missing ⚠️
internal/confmapprovider/discovery/config.go 0.00% 4 Missing ⚠️
internal/receiver/discoveryreceiver/receiver.go 50.00% 3 Missing and 1 partial ⚠️
tests/testutils/container.go 42.85% 4 Missing ⚠️
tests/testutils/endpoint.go 75.00% 4 Missing ⚠️
tests/testutils/socket_proxy.go 0.00% 4 Missing ⚠️
... and 13 more

❌ Your patch check has failed because the patch coverage (47.05%) is below the target coverage (85.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6974      +/-   ##
==========================================
- Coverage   38.01%   33.63%   -4.39%     
==========================================
  Files         367      366       -1     
  Lines       25713    20940    -4773     
==========================================
- Hits         9776     7044    -2732     
+ Misses      15124    13302    -1822     
+ Partials      813      594     -219     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@akhileshsingh85 akhileshsingh85 marked this pull request as ready for review November 25, 2025 23:22
@akhileshsingh85 akhileshsingh85 requested review from a team as code owners November 25, 2025 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant