[ISSUE #68]Change from pub fn to pub const fn for from_static_str method

[mxsm]

close #68

Summary by CodeRabbit

• New Features
  • from_static_str is now const, allowing creation of CheetahString from static strings in const contexts (e.g., consts, statics, and match patterns). This enables compile-time initialization and can reduce runtime overhead.
  • Existing code remains fully compatible; this change broadens where the API can be used without altering behavior at runtime.

[Copilot]

Pull Request Overview

Changes the from_static_str method from a regular public function to a const function, allowing it to be evaluated at compile time.

• Modified function declaration to use pub const fn instead of pub fn

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[coderabbitai]

Walkthrough

Changed CheetahString::from_static_str from a regular function to a const function in two impl locations, updating the signature to pub const fn from_static_str(s: &'static str) -> Self. No other logic or control flow changes.

Changes

Cohort / File(s)	Summary of edits
Const-qualify from_static_str src/cheetah_string.rs	Updated method signature(s) from pub fn from_static_str(...) -> Self to pub const fn from_static_str(...) -> Self in two impl blocks.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I twitch my ears at compile-time cheer,
A const now sparkles, crisp and clear.
Static strings hop straight to Self,
With zero runtime on the shelf.
Thump-thump—small change, swift delight,
Rusty stars wink through the night. 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title "[ISSUE #68]Change from pub fn to pub const fn for from_static_str method" succinctly summarizes the core update by indicating that the method signature has been changed to const, and it directly corresponds to the linked issue. It names the specific function and nature of the change, providing clear context at a glance. Including the issue number ties the title back to the project tracker without introducing extraneous detail.
Linked Issues Check	✅ Passed	The pull request updates the from_static_str method signature from pub fn to pub const fn exactly as required by issue #68, enabling its use in const contexts and matching the documented objective. The raw diff confirms no other behavioral or control flow changes were introduced, fulfilling the coding requirement of the linked issue. This aligns fully with the specified modification scope.
Out of Scope Changes Check	✅ Passed	The diff only alters the const-ness of the from_static_str method and does not introduce any unrelated modifications or additional features beyond the linked issue’s objective. All changes remain focused on enabling compile-time evaluation of that single method, indicating there are no out-of-scope edits. This keeps the pull request tightly scoped to the requested function signature update.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ 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 enh-68

---

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

src/cheetah_string.rs (1)

230-234: Add a const-context test for from_static_str

The change correctly makes from_static_str a const fn (it only constructs a struct from a &'static str). No existing tests exercise this in a const context—add one, for example:

const CS: CheetahString = CheetahString::from_static_str("hello");

#[test]
fn const_from_static_str() {
    assert_eq!(CS.as_str(), "hello");
}

Update the summary to note only a single const fn addition, not two locations.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2a5b8c1 and 993ea38.

📒 Files selected for processing (1)

• src/cheetah_string.rs (1 hunks)