व्यावहारिक Review Checklist

Chat नहीं, diff पढ़ें

Diff मायने रखता है, assistant का आत्मविश्वास भरा सार नहीं। बदली हुई हर पंक्ति पढ़ें, tests चलाएँ, और ख़ुद से पूछें: क्या यह असली अनुरोध हल करता है? क्या इसने कुछ ऐसा किया जो मैंने नहीं माँगा (अतिरिक्त files, config बदलाव, हटाए गए tests)? क्या मैं हर पंक्ति किसी साथी को समझा सकता हूँ? नहीं, तो commit से पहले उसे सरल करें या सीखें। उन्हीं hunks को stage करने के लिए, जिन्हें आप स्वीकारते हैं, git add -p उपयोग करें।

Checklist

इसे इतना छोटा रखें कि आप हर बार सचमुच उपयोग करें।

[ ] Solves the request I actually made (and nothing extra)
[ ] Every new function / method / package exists (checked docs)
[ ] Tests pass; at least one fails if I break the code
[ ] Edge cases: empty, one, many, invalid input
[ ] No secrets, no string-built SQL / shell commands, no disabled checks
[ ] No unrelated edits (config, deleted tests, formatting churn)
[ ] I can explain each line to a teammate

त्वरित जाँच: आपको क्या review करना चाहिए: assistant का सार या diff?

• स्वयं diff
• सिर्फ़ सार
• कोई नहीं
• Commit का समय