Skip to content

Add solution for Challenge 18 by onomica#1220

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

Add solution for Challenge 18 by onomica#1220
github-actions[bot] merged 2 commits intoRezaSi:mainfrom
onomica:challenge-18-onomica

Conversation

@onomica
Copy link
Copy Markdown
Contributor

@onomica onomica commented Jan 28, 2026

Challenge 18 Solution

Submitted by: @onomica
Challenge: Challenge 18

Description

This PR contains my solution for Challenge 18.

Changes

  • Added solution file to challenge-18/submissions/onomica/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 28, 2026

Walkthrough

Adds a new Go solution file for Challenge 18 containing temperature conversion utility functions. Implements CelsiusToFahrenheit, FahrenheitToCelsius, and Round functions, along with a main function demonstrating their usage with example temperature conversions and formatting.

Changes

Cohort / File(s) Summary
Temperature Conversion Utilities
challenge-18/submissions/onomica/solution-template.go
Added three exported functions: CelsiusToFahrenheit (applies formula C × 9/5 + 32 with 2-decimal rounding), FahrenheitToCelsius (applies formula (F - 32) × 5/9 with 2-decimal rounding), and Round (utility for rounding values to specified decimal places using math.Pow10 and math.Round). Includes main function with example usage.

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 75.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 summarizes the main change: adding a solution for Challenge 18 by onomica to the repository.
Description check ✅ Passed The description is directly related to the changeset, clearly explaining that it contains a Challenge 18 solution submission with relevant context about testing and code quality.

✏️ 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 3968418 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, @onomica!

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