Use a `Reporter` reference instead of a `ReporterImpl` reference in `ProjectAnalyzer`.
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 48/100
Research direction
Start with ProjectAnalyzer.java at the referenced reporter field and inspect the Reporter and ReporterImpl types. Then check the TestReport class in test-utils; the change is complete when ProjectAnalyzer uses the Reporter interface and TestReport implements Reporter without breaking the tests.
Written by the indexing model from the issue text.
Description
Current Limitation
Currently, our Reporter reference is of type ReporterImpl in the ProjectAnalyzer class here. However, this is not the ideal, given that we should allow external parties to override the behavior of the reporter to change the analyzer behavior.
Suggested Improvement
Alter the project analyzer source to have a reference to the Reporter interface. Need to change the TestReport class of test-utils to implement Reporter to match this change as well.
Version
No response
- Dominant language
- Java
- Stars
- 15
- Forks
- 12
- Avg merge
- 11h 57m
- Merged PRs (30d)
- 14
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ballerina-platform/static-code-analysis-tool
-
Type/Improvement
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
ballerina-platform/static-code-analysis-tool#140 ·
Maintainers usually reply within 1 day
-
Type/Improvement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
ballerina-platform/static-code-analysis-tool#139 ·
Maintainers usually reply within 1 day
-
Type/Improvement
Difficulty 3/5 1-2 days Newbie friendliness 62/100
ballerina-platform/static-code-analysis-tool#138 ·
Maintainers usually reply within 1 day
-
Type/Improvement
Difficulty 3/5 1-2 days Newbie friendliness 63/100
ballerina-platform/static-code-analysis-tool#137 ·
Maintainers usually reply within 1 day
-
Type/Improvement
Difficulty 3/5 1-2 days Newbie friendliness 62/100
ballerina-platform/static-code-analysis-tool#136 ·
Maintainers usually reply within 1 day
All issues in ballerina-platform/static-code-analysis-tool
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
open-metadata/OpenMetadata#36463 ·
Maintainers usually reply within 1 day
-
bug web-server
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
jooby-project/jooby#4035 ·
Maintainers usually reply within 1 day
-
afk-ok area:data-quality importer size:S
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
enorm-labs/event-junkie#4140 ·
Maintainers usually reply within 1 day
-
L: javascript
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
dependabot/dependabot-core#16744 ·
Maintainers usually reply within 1 day
-
[Security] NOTICE.md links to hijacked domain kineticjs.com, which now redirects to a betting siteOpen
Difficulty 1/5 Under an hour Newbie friendliness 83/100
eclipse-ee4j/jersey#6144 ·
Maintainers usually reply within 1 day