Add solution for Challenge 1 by Mwazowsky

[Mwazowsky]

Challenge 1 Solution

Submitted by: @Mwazowsky
Challenge: Challenge 1

Description

This PR contains my solution for Challenge 1.

Changes

• Added solution file to challenge-1/submissions/Mwazowsky/solution-template.go

Testing

• Solution passes all test cases
• Code follows Go best practices

Thank you for reviewing my submission! 🚀

[coderabbitai]

Walkthrough

Adds a new solution template file for Challenge 1 that implements a Sum function to add two integers and a main program that reads two comma-separated integers from standard input, computes their sum, and prints the result with basic error handling.

Changes

Cohort / File(s)	Summary
Challenge 1 Solution Submission challenge-1/submissions/Mwazowsky/solution-template.go	New Go program that defines an exported Sum(a int, b int) int function and a main function that reads two integers from stdin using fmt.Scanf with format "%d, %d", invokes Sum, and prints the result with error handling

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Single file addition with straightforward logic
• Simple arithmetic operation with no complex control flow
• Standard error handling pattern

Possibly related PRs

• Add solution for Challenge 1 by es-codigo #676: Adds identical Sum(a int, b int) int function and main logic reading two comma-separated integers in a different challenge-1 submission.
• Add solution for Challenge 1 by cckwes #844: Adds nearly identical Go solution for Challenge 1 with the same Sum function signature and "%d, %d" input parsing.
• Add solution for Challenge 1 by wangt95117 #832: Adds the same exported Sum(a int, b int) int function and similar solution-template program under challenge-1/submissions in a different contributor directory.

Pre-merge checks

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a solution for Challenge 1 by the contributor Mwazowsky, which matches the changeset.
Description check	✅ Passed	The description is directly related to the changeset, providing context about the Challenge 1 solution submission, the file location, and testing status.

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (2)

challenge-1/submissions/Mwazowsky/solution-template.go (2)

11-14: Consider improving error handling for better Go practices.

The current error handling is functional, but you might want to consider:

• Writing error messages to stderr using fmt.Fprintln(os.Stderr, ...)
• Exiting with a non-zero status code using os.Exit(1) to signal failure

This would make the program more robust if the challenge tests check exit codes.

Based on the search results, writing errors to os.Stderr instead of os.Stdout helps separate error messages from regular program output, and error messages should be printed to stderr using fmt.Fprintf(os.Stderr, ...).

Apply this diff to improve error handling:

+import (
+	"fmt"
+	"os"
+)
+
 func main() {
 	var a, b int
 	// Read two integers from standard input
 	_, err := fmt.Scanf("%d, %d", &a, &b)
 	if err != nil {
-		fmt.Println("Error reading input:", err)
-		return
+		fmt.Fprintf(os.Stderr, "Error reading input: %v\n", err)
+		os.Exit(1)
 	}

---

21-24: LGTM! The Sum function is correct.

The implementation is straightforward and correct. As a minor style note, Go allows you to simplify the parameter declaration when both parameters share the same type.

Optional: Apply this diff for more idiomatic Go:

-func Sum(a int, b int) int {
+func Sum(a, b int) int {
 	return a + b
 }

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between daf4fa9 and 06f2d2f.

📒 Files selected for processing (1)

• challenge-1/submissions/Mwazowsky/solution-template.go (1 hunks)