style: adhere to new clippy lint warning

[2bndy5]

Seems to be related to changes in lint rules for Rust v1.93. See unnecessary_unwrap lint rule.

Summary by CodeRabbit

• Refactor
  • Improved internal error handling in GitHub REST API integration by using more robust pattern matching and default value strategies. No changes to end-user functionality or behavior.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Refactored error handling in the GitHub REST API module by replacing direct unwrap() calls with pattern matching for repository binding and using unwrap_or_default() for SHA retrieval, improving code safety while maintaining existing control flow behavior.

Changes

Cohort / File(s)	Summary
GitHub REST API Error Handling cpp-linter/src/rest_api/github/mod.rs	Refactored get_list_of_changed_files to use pattern matching (let Some(repo)) instead of is_some() with unwrap, changed SHA retrieval from unwrap() to unwrap_or_default(), and simplified URL construction with the bound repo variable.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

rust

🚥 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
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'style: adhere to new clippy lint warning' accurately summarizes the main change—addressing a new Clippy lint warning (unnecessary_unwrap) introduced in Rust v1.93.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-clippy-warning

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!

---

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: 1

🤖 Fix all issues with AI agents

In `@cpp-linter/src/rest_api/github/mod.rs`:
- Line 133: The code currently uses self.sha.clone().unwrap_or_default(), which
silently supplies an empty SHA and can produce invalid commit URLs; change this
to either assert presence or return an error: replace unwrap_or_default() with
unwrap() or expect("missing SHA for non-PR commit URL") to fail fast when SHA
must exist, or match on self.sha (e.g., if let Some(sha) = &self.sha { ... }
else { return Err(...) }) to explicitly handle the None case; also make the
treatment consistent with the other usage that does self.sha.clone().unwrap() so
both code paths either validate/expect SHA or both defensively return an error.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.04%. Comparing base (4c4abbb) to head (c71e6b2).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
+ Coverage   96.88%   97.04%   +0.15%     
==========================================
  Files          14       14              
  Lines        3118     3078      -40     
==========================================
- Hits         3021     2987      -34     
+ Misses         97       91       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.