andy icon indicating copy to clipboard operation
andy copied to clipboard

Refactor getNumberOfPassedChecks method

Open martinmladenov opened this issue 2 years ago • 2 comments

The getNumberOfPassedChecks in CodeChecksResult method currently accepts a boolean.

One way to avoid this boolean would be to offer two methods weightedNumberOfPassedChecks.

See this blog post explaining this Clean Code rule: https://www.informit.com/articles/article.aspx?p=1392524

Originally posted by @mauricioaniche in https://github.com/cse1110/andy/pull/210#discussion_r1225873677

martinmladenov avatar Jun 21 '23 10:06 martinmladenov

I'd love to do this issue

vesk4000 avatar Jul 05 '23 09:07 vesk4000

Thanks, I assigned you. Let me know if you have any questions :)

martinmladenov avatar Jul 05 '23 09:07 martinmladenov