Document proofs include value for get_sheet#161
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe ChangesSheet API documentation
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Coverage Report for CI Build 29828647042Warning No base build found for commit Coverage: 77.908%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
Adds
proofsto the documented validincludevalues forget_sheet, completing the proofs work started in #159.The
Proofmodel,Row.proof, and theget_reportdocstring already listedproofs(shipped in #159); theget_sheetdocstring was the remaining gap.includeis passed through, soinclude=['proofs']already worked at runtime — this is a docs-only fix.DEVECO-2197
Summary by CodeRabbit
Sheets.get_sheetdocumentation to listproofsas a valid include option.