Skip to content

Conversation

@naveenthangaraj03
Copy link
Contributor

@naveenthangaraj03 naveenthangaraj03 commented Aug 30, 2025

Closes #

📑 Description

ISSUES:

  • Flag filter is not supported for custom-analyzer.
  • If we have more than one custom analyzer need to run separate server for each custom analyzer.

RESOLVED:

  • Flag filter is added for custom-analyzer, so we can define a lot of custom analyzer in a single server and run specific custom analyzer based on filter.
image

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

ℹ Additional Information

Signed-off-by: naveenthangaraj03 <[email protected]>
@codecov
Copy link

codecov bot commented Aug 30, 2025

Codecov Report

❌ Patch coverage is 39.43662% with 43 lines in your changes missing coverage. Please review.
✅ Project coverage is 44.80%. Comparing base (173e4dc) to head (ff47060).
⚠️ Report is 189 commits behind head on main.

Files with missing lines Patch % Lines
pkg/analysis/analysis.go 38.70% 36 Missing and 2 partials ⚠️
cmd/analyze/analyze.go 0.00% 5 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1574       +/-   ##
===========================================
+ Coverage   34.76%   44.80%   +10.03%     
===========================================
  Files          94      114       +20     
  Lines        6342     8356     +2014     
===========================================
+ Hits         2205     3744     +1539     
- Misses       4046     4423      +377     
- Partials       91      189       +98     

☔ 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.

@naveenthangaraj03
Copy link
Contributor Author

@AlexsJones, Can you take a look into this PR

@naveenthangaraj03
Copy link
Contributor Author

@AlexsJones , does this PR looks good?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Proposed

Development

Successfully merging this pull request may close these issues.

2 participants