Repository navigation
docs(agents): check documentation before opening a pull request - #130
Conversation
Record which docs can go stale (docs/user, docs/dev, README, and the product docs in deepnote/deepnote) so they are updated alongside the change, and ask for small single-purpose pull requests. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughAdded guidance in Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Other Merge Risk: ⚪ Minimal · up to This change adds contributor documentation guidance only and does not alter product behavior. It is safe to merge. 🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
Comment |
|
📦 Python package built successfully!
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #130 +/- ##
==========================================
- Coverage 77.26% 77.09% -0.17%
==========================================
Files 115 115
Lines 6589 6589
Branches 961 961
==========================================
- Hits 5091 5080 -11
- Misses 1186 1198 +12
+ Partials 312 311 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. |
|
🚀 Review App Deployment Started
|
Addresses review feedback: the guidance was subjective, unrelated to the documentation check this change is about, and duplicated the objection already raised on deepnote/tf-infra#493. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
37a2e1f
Adds a Documentation section to
CLAUDE.md: check whether a change makes documentation stale before opening a PR, with the places it lives —docs/user/,docs/dev/,README.md,CONTRIBUTING.md, and, for anything a Deepnote user sees,docs/in the publicdeepnoterepository, published at https://deepnote.com/docs.Part of a small pass applying the same rule across the Deepnote repos.
The Pull Requests section this PR originally also added has been dropped per review (#discussion_r4092320040), matching the same call on deepnote/tf-infra#493.
🤖 Generated with Claude Code
Summary by CodeRabbit