br(v28): mv noContent —> noBodySchema#3303
Conversation
|
Reviewed PR #3303 — clean rename of Task list (4/4 completed)
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Reviewed — no issues found.
The rename is complete with no stale references to noContent outside migration/ and CHANGELOG.md (which correctly reference the old name). The migration rule and its tests follow established patterns. The noContentFactory in example/ is a different concept (HTTP 204 endpoint factory) and is unaffected.
Minor observation: shouldHaveContent in common-helpers.ts still uses "Content" terminology while this PR shifts toward "Body" (noBodySchema, hasBody). Not blocking — renaming that helper would be a broader change touching documentation-helpers.ts and tests.
Claude Opus | 𝕏
There was a problem hiding this comment.
Reviewed — no issues found. The new commits are cosmetic changelog formatting only.
Claude Opus | 𝕏

Renaming for clarity