Skip to content

Add solution for Challenge 3 by Huansock#1226

Merged
github-actions[bot] merged 2 commits intoRezaSi:mainfrom
Huansock:challenge-3-Huansock
Apr 14, 2026
Merged

Add solution for Challenge 3 by Huansock#1226
github-actions[bot] merged 2 commits intoRezaSi:mainfrom
Huansock:challenge-3-Huansock

Conversation

@Huansock
Copy link
Copy Markdown
Contributor

Challenge 3 Solution

Submitted by: @Huansock
Challenge: Challenge 3

Description

This PR contains my solution for Challenge 3.

Changes

  • Added solution file to challenge-3/submissions/Huansock/solution-template.go

Testing

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

Thank you for reviewing my submission! 🚀

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 29, 2026

Walkthrough

A new Go program defining Employee and Manager types with in-memory employee management. The Manager type provides methods to add, remove, find, and calculate average salary of employees. A main function demonstrates typical usage patterns including employee addition, removal, and lookups.

Changes

Cohort / File(s) Summary
Employee Manager Solution
challenge-3/submissions/Huansock/solution-template.go
Adds Employee struct with ID, Name, Age, Salary fields; Manager struct with slice of employees; methods to add/remove employees, calculate average salary, and find employee by ID; includes demo main function.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • #1099: Implements identical Employee and Manager types with the same set of methods (AddEmployee, RemoveEmployee, GetAverageSalary, FindEmployeeByID) in the same challenge-3 solution file.
  • #1118: Adds near-identical Go implementations with Employee and Manager types and the same manager methods plus a similar main demonstration.
  • #879: Implements the same Employee and Manager types and identical Manager methods in the challenge-3 solution file structure.
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding a solution file for Challenge 3 by the author Huansock.
Description check ✅ Passed The description is directly related to the changeset, explaining the Challenge 3 solution submission with relevant details about the file location and testing claims.

✏️ 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.

@github-actions github-actions bot merged commit 40a7b96 into RezaSi:main Apr 14, 2026
6 checks passed
@github-actions
Copy link
Copy Markdown

🎉 Auto-merged!

This PR was automatically merged after 2 days with all checks passing.

Thank you for your contribution, @Huansock!

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.

2 participants