Add inclusion and refactor exclusion - Githubissues

debricked / cli

Debricked's command line interface. It brings open source security, compliance and health to your project via the command prompt.

MIT License

19 stars 8 forks source link

Add inclusion and refactor exclusion #217

Closed filip-debricked closed 2 months ago

sweoggy commented 3 months ago

It has been decided that this will go into the 2.0 release instead

sweoggy commented 2 months ago

@filip-debricked could you please add to UPGRADE-2.0.md mentioning the behaviour change caused by this PR?

github-actions[bot] commented 2 months ago

CodeScene PR Check

✅ Code Health Quality Gates: OK

Declining Code Health: 8 findings(s) 🚩
Improving Code Health: 5 findings(s) ✅
Affected Hotspots: 6 files(s) 🔥

View detailed results in CodeScene

🚩 Declining Code Health (highest to lowest):

Large Method scan.go: NewScanCmd 🔥
Large Method fingerprint_test.go: TestShouldProcessFile 🔥
Large Method finder_test.go: TestGetGroupsWithStrictFlag 🔥
Excess Number of Function Arguments strategy_factory.go: Make
Complex Method resolve.go: NewResolveCmd 🔥
Excess Number of Function Arguments strategy_mock_factory.go: Make
Excess Number of Function Arguments strategy_mock_factory.go: Make
Bumpy Road Ahead exclusion.go: Excluded 🔥

✅ Improving Code Health:

Primitive Obsession fingerprint.go 🔥
String Heavy Function Arguments fingerprint.go 🔥
Primitive Obsession resolver.go
Complex Method fingerprint.go: FingerprintFiles 🔥
Bumpy Road Ahead fingerprint.go: FingerprintFiles 🔥