epam / epam/Gepard

Java CC review: Address findings

Open
#3 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Java
Stars
25
Forks
42
PR merge metrics
No merged PRs in 30d

Description

At the time this issue was created, we have:
- around 122 issues detected by Sonar - need to be reduced constantly
- Unit test coverage is about 35%, need to be increased
- fake properties in gradle.properties file, that will influence all local repos, need to remove them and use -P, --project-props instead
- check the usage of catching Throwables, and keep only where it is a must (catch only exceptions we handle, and exceptions those are necessary to catch)
- remove checkstyle supression on method names (that allows underscores)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the Sonar findings, unit-test coverage, gradle.properties, Throwable catches, and the checkstyle suppression mentioned in the issue. Done means the applicable findings are reduced, coverage is increased, fake properties are removed, catches are narrowed where appropriate, and the method-name suppression is removed.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system, testing, tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.