Create simple_utils.py

[mitsuomkinoshita-tech]

Summary by CodeRabbit

• New Features
  • Added lightweight utilities for reversing text, counting words in a sentence, and converting temperatures from Celsius to Fahrenheit to support simple text manipulation and temperature display.

[coderabbitai]

Walkthrough

Adds a new module python/simple_utils.py containing three standalone utility functions: reverse_string, count_words, and celsius_to_fahrenheit. No other files or integrations were changed.

Changes

Cohort / File(s)	Summary of Changes
Utility functions python/simple_utils.py	Added three functions: reverse_string(text) (returns the reversed string), count_words(sentence) (counts words by splitting on whitespace), and celsius_to_fahrenheit(celsius) (converts Celsius to Fahrenheit using (c * 9/5) + 32).

Sequence Diagram(s)

sequenceDiagram
    participant Caller as Caller
    participant Utils as python/simple_utils.py
    Note over Utils: New utility functions
    Caller->>Utils: reverse_string(text)
    Utils-->>Caller: reversed_text
    Caller->>Utils: count_words(sentence)
    Utils-->>Caller: word_count
    Caller->>Utils: celsius_to_fahrenheit(celsius)
    Utils-->>Caller: fahrenheit_value

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I nibbled at strings and turned them around,
Counted soft words where white spaces are found.
From frosty to toasty I hopped with a spin,
Degrees translated — a warm little grin.
Thump-thump, utilities stitched; a small rabbit win! 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% 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
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title precisely reflects the main change—creating the simple_utils.py module—and is concise, clear, and directly tied to the pull request’s purpose.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 472d5bc and 5690f1f.

📒 Files selected for processing (1)

• python/simple_utils.py (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• python/simple_utils.py

---

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 (3)

python/simple_utils.py (3)

3-5: Consider adding type hints and handling edge cases.

The function works correctly for string inputs, but consider the following improvements:

• Add type hints for better code documentation and IDE support.
• Consider handling edge cases like None or non-string inputs.

Apply this diff to add type hints:

-def reverse_string(text):
+def reverse_string(text: str) -> str:
   """Reverses the characters in a string."""
   return text[::-1]

If you want to add basic input validation:

-def reverse_string(text):
-  """Reverses the characters in a string."""
+def reverse_string(text: str) -> str:
+  """Reverses the characters in a string.
+  
+  Args:
+    text: The string to reverse.
+    
+  Returns:
+    The reversed string.
+  """
+  if not isinstance(text, str):
+    raise TypeError(f"Expected str, got {type(text).__name__}")
   return text[::-1]

---

7-8: Add docstring and type hints for consistency.

The function is missing a docstring, which is inconsistent with reverse_string. Additionally, type hints would improve code clarity.

Apply this diff:

-def count_words(sentence):
+def count_words(sentence: str) -> int:
+  """Counts the number of words in a sentence.
+  
+  Args:
+    sentence: The sentence to count words in.
+    
+  Returns:
+    The number of words (split on whitespace).
+  """
   return len(sentence.split())

---

10-11: Add docstring and type hints for consistency.

The function is missing a docstring and type hints. Since temperature conversion can work with both integers and floats, consider using numeric type hints.

Apply this diff:

-def celsius_to_fahrenheit(celsius):
+def celsius_to_fahrenheit(celsius: float) -> float:
+  """Converts Celsius temperature to Fahrenheit.
+  
+  Args:
+    celsius: Temperature in Celsius.
+    
+  Returns:
+    Temperature in Fahrenheit.
+  """
   return (celsius * 9/5) + 32

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 7abc6b5 and 472d5bc.

📒 Files selected for processing (1)

• python/simple_utils.py (1 hunks)