Col-E / Col-E/BentoFX

Static Code Analysis and Quality Gates

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

Description

@Col-E, how do you feel about modifying the GitHub workflows to include static code analysis?

For background, I added analysis steps for [SonarQube Cloud](https://sonarcloud.io), [Qodana](https://qodana.cloud), and [CodeQL](https://codeql.github.com/) in the persistence fork, and they've been very helpful in pointing out potential bugs, potential NPEs, gaps in test coverage, etc. I've already fixed the issues discovered in the persistence modules, but there are still issues in the `core` module. Analysis in the fork is configured to ignore the `core` and `demos/basic` modules, so porting the configuration **shouldn't** break anything. Those modules can be included as findings in them are resolved. I'd be happy to open issues for them and implement most as separate PRs.

Adopting the changes would require creating a [SonarQube Cloud](https://sonarcloud.io) account and generating an access token. [Codecov](https://codecov.io) and [Qodana Cloud](https://qodana.cloud) accounts would be needed as well, each with its own token. So we'd likely want to coordinate closely to avoid downtime. I don't want to "move any cheese," but I'm willing to port them over if it's something you actually want. Let me know if you're interested.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the repository's GitHub workflows and the static-analysis configuration from the persistence fork, noting how the core and demos/basic modules are excluded. Check the required SonarQube Cloud, Qodana Cloud, Codecov, and CodeQL setup and credentials; done means the analysis workflows and quality gates run reliably without downtime.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions, java
Domain
ci-cd, security, tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.