Walkthrough

Documentation comments were removed from six PyStr methods (removeprefix, removesuffix, format_map, title, isprintable, istitle) in the builtins module without modifying their implementations or behavior.

Changes

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• No functional logic changes; purely documentation removal
• Consistent pattern applied across multiple method declarations
• No risk of introducing bugs or altering behavior

Poem

A rabbit hops through comments old,
Where docs once spoke, now silence gold—
Six methods bare, their words now freed,
Simplicity is all we need! 🐰✨

Pre-merge checks and finishing touches

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.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

@ShaharNaveh I forgot what was the major reason to split it out. Probably the license and workflow part. To integrate it again, guidelines:

• Please check the generate script is still working well.
• The license must be aligned to CPython
• We'd better to generate them and commit from CI. (manual trigger when upgrading version)

@ShaharNaveh I forgot what was the major reason to split it out. Probably the license and workflow part. To integrate it again, guidelines:

• Please check the generate script is still working well.

I intend to refractor it, either way I'll make it will work.

• The license must be aligned to CPYTHON

Sure, will copy CPython license to crates/rustpython_doc_db/LICENSE

• We'd better to generate them and commit from CI. (manual trigger when upgrading version)

I thought of having the generate script to have a --check flag that will exit with non-zero on any diff, we will check it on the CI.

I thought of having the generate script to have a --check flag that will exit with non-zero on any diff, we will check it on the CI.

The current script is depends on python runtime. Each platform generate their own set of document. To cover the entire platform, the results from different platforms must be merged.

I thought of having the generate script to have a --check flag that will exit with non-zero on any diff, we will check it on the CI.

The current script is depends on python runtime. Each platform generate their own set of document. To cover the entire platform, the results from different platforms must be merged.

Make sense, I don't expect nt to be available on Linux. But I don't think that there's a difference between docstring of same object on different platforms, right? i.e, str.count is the same on Windows, Linux, MacOS. correct?

Yes, they are.