[GH-ISSUE #83] Cleanup code issues on go report card #58

New issue

Closed

opened 2026-03-03 16:22:01 +03:00 by kerem · 1 comment

kerem commented

2026-03-03 16:22:01 +03:00

Owner

Originally created by @mum4k on GitHub (Jan 22, 2019).
Original GitHub issue: https://github.com/mum4k/termdash/issues/83

https://goreportcard.com/report/github.com/mum4k/termdash

Note: not all of the gocyclo issues should be cleaned up, some functions need to have those switch statements with many cases (like selecting a character). Similarly we don't really want to shorten tests. So the gocyclo category needs judgement.

The other categories are straightforward.

Originally created by @mum4k on GitHub (Jan 22, 2019). Original GitHub issue: https://github.com/mum4k/termdash/issues/83 https://goreportcard.com/report/github.com/mum4k/termdash Note: not all of the gocyclo issues should be cleaned up, some functions need to have those switch statements with many cases (like selecting a character). Similarly we don't really want to shorten tests. So the gocyclo category needs judgement. The other categories are straightforward.

kerem

2026-03-03 16:22:01 +03:00

closed this issue
added the
good first issue

help wanted

cleanup
labels

kerem commented

2026-03-03 16:22:02 +03:00

Author

Owner

@nilathedragon commented on GitHub (Jan 22, 2019):

I've created a pull-request #84 which fixes all except the go-cyclo errors

@nilathedragon commented on GitHub (Jan 22, 2019): I've created a pull-request #84 which fixes all except the go-cyclo errors

kerem referenced this issue

2026-03-03 16:23:09 +03:00

[PR #58] [MERGED] Ability to copy one canvas onto another. #177