Skip to content

Add solution for Challenge 3 by MaryNfs#1260

Open
MaryNfs wants to merge 1 commit intoRezaSi:mainfrom
MaryNfs:challenge-3-MaryNfs
Open

Add solution for Challenge 3 by MaryNfs#1260
MaryNfs wants to merge 1 commit intoRezaSi:mainfrom
MaryNfs:challenge-3-MaryNfs

Conversation

@MaryNfs
Copy link

@MaryNfs MaryNfs commented Feb 4, 2026

Challenge 3 Solution

Submitted by: @MaryNfs
Challenge: Challenge 3

Description

This PR contains my solution for Challenge 3.

Changes

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

Testing

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

Thank you for reviewing my submission! 🚀

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 4, 2026

Walkthrough

This PR introduces a new Go solution file implementing Employee and Manager types with methods to add employees, remove by ID, calculate average salary, and find employees by ID. The Manager stores a slice of Employee records and provides operations to manipulate and query them.

Changes

Cohort / File(s) Summary
Challenge 3 Solution
challenge-3/submissions/MaryNfs/solution-template.go
Introduces Employee and Manager types with four manager methods: AddEmployee() appends to the employee slice, RemoveEmployee() removes by ID using slices.Delete, GetAverageSalary() computes mean salary (returns 0 for empty list), and FindEmployeeByID() returns a pointer to the matching employee or nil.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

🚥 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 reflects the main change: adding a Challenge 3 solution by MaryNfs to the repository.
Description check ✅ Passed The description is directly related to the changeset, explaining that it contains a Challenge 3 solution submitted by MaryNfs with clear details about what was added and testing claims.

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

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


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