Skip to content

Ricecooker resolve duplicative documentation and methods#648

Merged
LianaHarris360 merged 1 commit intomainfrom
claude/review-issue-24-01XMhMz8sqLuSqBffHZPdQ4Y
Feb 10, 2026
Merged

Ricecooker resolve duplicative documentation and methods#648
LianaHarris360 merged 1 commit intomainfrom
claude/review-issue-24-01XMhMz8sqLuSqBffHZPdQ4Y

Conversation

@rtibbles
Copy link
Copy Markdown
Member

@rtibbles rtibbles commented Dec 17, 2025

Summary

  • Document all shared attributes in Node base class
  • Simplify subclass docstrings to reference parent classes
  • Refactor ContentNode.to_dict() to call super()
  • Remove redundant init methods in ContentNode subclasses

References

Closes #24

Reviewer guidance

Check that all tests still pass, and that the documentation matches the arguments.

🤖 This was created by Claude Code. @rtibbles then reviewed the generated output, and did iterative rounds of updates before making it ready for review 🤖

- Document all shared attributes in Node base class
- Simplify subclass docstrings to reference parent classes
- Refactor ContentNode.to_dict() to call super()
- Remove redundant __init__ methods in ContentNode subclasses

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@LianaHarris360 LianaHarris360 merged commit 78a2ec7 into main Feb 10, 2026
40 of 45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reduce duplication on ContentNode subclasses

3 participants