Skip to content

Add solution for Challenge 1#1244

Open
DEEZY4U wants to merge 2 commits intoRezaSi:mainfrom
DEEZY4U:main
Open

Add solution for Challenge 1#1244
DEEZY4U wants to merge 2 commits intoRezaSi:mainfrom
DEEZY4U:main

Conversation

@DEEZY4U
Copy link

@DEEZY4U DEEZY4U commented Jan 31, 2026

No description provided.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 31, 2026

Walkthrough

Adds two new Go solution files: one in challenge-1/submissions/DEEZY4U/solution-template.go that reads two comma-separated integers, defines Sum(a int, b int) int, and prints the sum with basic error handling; and one in challenge-2/submissions/DEEZY4U/solution-template.go that reads a string, defines ReverseString(s string) string, and prints the reversed string.

Changes

Cohort / File(s) Summary
Challenge 1 Solution
challenge-1/submissions/DEEZY4U/solution-template.go
New file: main reads two comma-separated ints via fmt.Scanf("%d, %d"), prints errors on read failure, and exposes func Sum(a int, b int) int returning a + b.
Challenge 2 Solution
challenge-2/submissions/DEEZY4U/solution-template.go
New file: main reads a line from stdin and prints the result of exported func ReverseString(s string) string, which reverses the input by converting to runes and swapping two pointers.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

🚥 Pre-merge checks | ❌ 3
❌ Failed checks (2 warnings, 1 inconclusive)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title only mentions Challenge 1, but the changeset includes solutions for both Challenge 1 and Challenge 2, making it incomplete. Update the title to reflect both challenges, e.g., 'Add solutions for Challenge 1 and Challenge 2' or 'Add Challenge 1 and Challenge 2 solutions'.
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive No pull request description was provided by the author. Add a description explaining the changes, even if brief, to help reviewers understand the intent and context of these solutions.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant