archethic-foundation / archethic-foundation/archethic-node

Take into consideration sobelow warnings

Open
#641 1 comment 0 reactions 0 assignees View on GitHub
CI/CD quality
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.