Add solution for Challenge 22 by alimkinpark#1651
Conversation
WalkthroughAdds a complete Go solution for the coin-change problem using a greedy algorithm. Includes a main function that tests with U.S. coin denominations, and two exported functions: ChangesCoin-Change Solution
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (2.11.4)level=error msg="[linters_context] typechecking error: pattern ./...: directory prefix . does not contain main module or its selected dependencies" Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
challenge-22/submissions/alimkinpark/solution-template.go (1)
32-32: 💤 Low valueRemove leftover
TODOstubs.Both
// TODO: Implement this functioncomments are template artifacts — the functions are fully implemented.🧹 Proposed cleanup
func MinCoins(amount int, denominations []int) int { - // TODO: Implement this function count := 0func CoinCombination(amount int, denominations []int) map[int]int { - // TODO: Implement this function result := make(map[int]int)Also applies to: 54-54
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 4f8c352e-b5f6-4930-92ae-a503e8b89356
📒 Files selected for processing (1)
challenge-22/submissions/alimkinpark/solution-template.go
Challenge 22 Solution
Submitted by: @alimkinpark
Challenge: Challenge 22
Description
This PR contains my solution for Challenge 22.
Changes
challenge-22/submissions/alimkinpark/solution-template.goTesting
Thank you for reviewing my submission! 🚀