search

CoderanchCorral / Blackjack

Mozilla Public License 2.0
17 stars 2 forks source link

Main application loop to build a hand of cards #32

Closed aquino-a closed 2 years ago

aquino-a commented 3 years ago

resolves #25

aquino-a commented 3 years ago

Does the main application really need code coverage?

nibsi commented 3 years ago

Does the main application really need code coverage?

What makes it so special that it doesn't need testing?

nibsi commented 3 years ago

Please reopen this PR when the InputUtility and the Hand PRs have merged. It's getting a bit muddy here.

sonarcloud[bot] commented 3 years ago

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

82.8% 82.8% Coverage
0.0% 0.0% Duplication

nibsi commented 2 years ago

I updated the code analysis rules. There are 4 code smells left, but you don't have to fix them. I want to overhaul and clean up some stuff after this PR has been merged.

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

85.9% 85.9% Coverage
0.0% 0.0% Duplication