archethic-foundation / archethic-foundation/archethic-node
Take into consideration sobelow warnings
- Dominant language
- Elixir
- Stars
- 82
- Forks
- 24
- PR merge metrics
- No merged PRs in 30d
Description
### Is your feature request related to a problem?
Sobelow as been integrated into CI with PR #631, hence we should take into consideration the warnings returned by it.
### Describe the solution you'd like
Sobelow provide 3 confidence levels (low, medium, high).
We should fix the warning at least for medium and high confidence.
We should add an option to exit the process with an error if medium or high confidence warnings are returned to fail the CI test using option `--exit medium`.
Low confidence level may be corrected but should not be blocking.
### Additional context
_No response_
Contributor guide
Research direction
Review PR #631 and the Sobelow results produced by CI first. Identify the medium- and high-confidence warnings that remain, then confirm how the CI command is configured. Done means those warnings are addressed and Sobelow runs with `--exit medium`, while low-confidence warnings do not block CI.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- elixir
- Domain
- ci-cd, security
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100