पाठ 21 / 26
Checklist और Review Skills
टीम के मानकों को दोहराने योग्य समीक्षा प्रक्रिया के रूप में कूटबद्ध करें।
मानक जो ख़ुद लागू हों
Skills का सबसे मूल्यवान उपयोग टीम के अलिखित मानकों को प्रक्रिया में बदलना है: pull-request review skill जो diff को आपकी checklist (error सँभाल, tests, नामकरण, सुरक्षा) के विरुद्ध चलती है, database-migration जाँच (locks, backward compatibility, rollback), accessibility audit, pre-release तैयारी रिपोर्ट। पैटर्न: references/ में checklist file, रिपोर्ट template वाली केवल-पढ़ने वाली skill, और नियम कि हर निष्कर्ष प्रमाण (file और पंक्ति) उद्धृत करे ताकि असत्यापित दावे हट जाएँ। Checklist छोटी और अपने codebase की असली आवर्ती समस्याओं पर विशिष्ट रखें; सामान्य 60-बिंदु सूची अनदेखी की जाती है। Review skills को केवल-पढ़ने वाली (संपादन नहीं) रखें ताकि कहीं भी चलाना सुरक्षित हो, और मनुष्य के निष्कर्ष पढ़ने के बाद सुधार लगाने का अलग चरण रखें।
लिखने योग्य Skills
सबसे उपयोगी skills checklists, templates, सत्यापन loops और टीम playbooks हैं।
समीक्षा रिपोर्ट template
हर निष्कर्ष के लिए file और पंक्ति माँगना रिपोर्ट को ईमानदार रखता है। उदाहरण; यहाँ चलाया नहीं गया।
# PR review: <title>
## Summary
One paragraph: what the PR does and whether it matches the description.
## Findings
| Severity | File:line | Problem | Suggested fix |
|---|---|---|---|
| high | shop/cart.py:42 | discount applied before tax rounding | round after tax |
## Checklist
- [x] tests added/updated
- [ ] error paths handled
- [x] no secrets or debug code
Rule: every row cites a real file:line from the diff. Delete rows you cannot cite.Review skills केवल-पढ़ने वाली रखें
जो skill संपादन नहीं कर सकती वह कहीं भी चलाना सुरक्षित है; सुधार अलग, अनुमोदित चरण में लगाएँ।
त्वरित जाँच: हर समीक्षा निष्कर्ष को file और पंक्ति क्यों उद्धृत करनी चाहिए?
- यह रिपोर्ट लंबी करता है
- यह प्रमाण को मजबूर करता है और असत्यापित दावों को हटाने देता है
- यह Git के लिए ज़रूरी है
- निष्कर्षों को प्रमाण नहीं चाहिए
Answer
यह प्रमाण को मजबूर करता है और असत्यापित दावों को हटाने देता है — उद्धृत प्रमाण जाँचा जा सकता है; अस्पष्ट दावे नहीं।