Pull requests र code review के हुन्?

Question

Accepted Answer

एक **pull request** (PR) — वा merge request — अर्को शाखामा परिवर्तनहरू merge गर्नको लागि एक अनुरोध हो, जसले **code review**, छलफल, र integration भन्दा पहिले स्वचालित जाँचहरूको लागि एक स्थान प्रदान गर्दछ। PRहरू आधुनिक सहयोगी विकास र कोड गुणस्तरको केन्द्रबिन्दु हुन्।

## Pull request के हो

```text
A PR proposes merging a branch (e.g. a feature) into another (e.g. main):
  → shows the DIFF (what changed), a description, and discussion
  → reviewers comment, request changes, or approve
  → CI runs automated checks (tests, linting, builds) on the PR
  → once approved + checks pass → MERGE into the target branch
→ It's a gate and collaboration point before code is integrated.
```

## PR workflow

```text
1. Create a feature branch; make commits; push it
2. Open a PR (feature → main) with a clear description of what/why
3. Automated CHECKS run (tests, lint, build) — must pass
4. REVIEWERS examine the code: comment, suggest, approve, or request changes
5. Author addresses feedback (more commits)
6. Approved + checks green → MERGE (merge commit, squash, or rebase)
```

## Code review किन महत्त्वपूर्ण छ

```text
✓ QUALITY — catches bugs, design issues, and edge cases before they reach main
✓ KNOWLEDGE SHARING — others learn the code; the author gets feedback; spreads context
✓ CONSISTENCY — enforces standards, conventions, best practices
✓ SHARED OWNERSHIP — more than one person understands each change
✓ MENTORSHIP — a venue for learning and improving
```

## राम्रो PR अभ्यासहरू

```text
✓ Keep PRs SMALL and focused (easier, better reviews — huge PRs get rubber-stamped)
✓ Clear title and description (what, why, how to test)
✓ Respond constructively to feedback; review others' PRs thoughtfully and kindly
✓ Ensure CI passes before requesting review
```

## किन यो महत्त्वपूर्ण छ

Pull requests र code review बुझ्नु महत्त्वपूर्ण छ किनभने तिनीहरू **आधुनिक सहयोगी विकास र कोड गुणस्तरको केन्द्रबिन्दु** हुन्, त्यसैले यो व्यावसायिक टिम कामको लागि मूल्यवान व्यावहारिक ज्ञान हो।

एक **pull request** परिवर्तनहरू प्रस्ताव गर्न र एकीकरण गर्नको लागि मानक तन्त्र हो — diff हेर्न, छलफल गर्न, स्वचालित जाँचहरू चलाउन, र merge गर्नु अघि कोड समीक्षा गर्नको लागि एक स्थान प्रदान गर्दछ, गुणस्तरको गेट र सहयोग बिन्दुको रूपमा काम गर्दछ।

**PR workflow** (feature branch बनाउँदै, स्पष्ट विवरण सहित PR खोल्दै, स्वचालित CI जाँचहरू चलिरहेका छन्, reviewers परीक्षण र प्रतिक्रिया दिइरहेका छन्, लेखकले यसलाई सम्बोधन गर्दै छन्, र अनुमोदित र हरियो भएपछी merge गर्दै) बुझ्नु आवश्यक छ किनभने यो workflow ले विशाल बहुसंख्यक व्यावसायिक टिमहरूले कोड एकीकरण गरेका छन् — लगभग सर्वव्यापी प्रक्रिया।

sमालोत गरेर, **code review किन महत्त्वपूर्ण छ भनी** बुझ्नु मूल्यवान छ: यसले **गुणस्तर** सुधार गर्दछ (त्रुटिहरू, डिजाइन समस्याहरू, र edge cases मुख्य पहुँचनु अघी पक्राउ गर्दछ), **ज्ञान साझेदारी** सक्षम गर्दछ (अन्यहरू कोड, प्रसंग सिकन्छन्, लेखकले प्रतिक्रिया पाइन्छ), **स्थिरता** लागू गर्दछ (मान्यताहरू र अभ्यासहरू), **साझा स्वामित्व** बनाउँदछ, र **mentorship** प्रदान गर्दछ — code review लाई कोड गुणस्तर र टिम स्वास्थ्य दुवैको लागि सबैभन्दा मूल्यवान अभ्यासहरू मध्ये एक बनाइदै।

**राम्रो PR अभ्यासहरू** (PRहरू साना र प्रभावकारी समीक्षाको लागि केन्द्रित राख्दै, किनभने विशाल PRहरू rubber-stamp हुन्छन्; स्पष्ट विवरणहरू; रचनात्मक प्रतिक्रिया) व्यावसायिक सहयोग कौशल प्रतिबिम्बित गर्दछ।

Pull requests र code review आधुनिक टिमहरूले सफ्टवेयर विकास गर्ने तरिकाको लागि केन्द्रीय हुन् (गुणस्तर नियन्त्रण र सहयोगको साथ परिवर्तनहरू एकीकरणको लागि मानक), र workflow, समीक्षाको मान्यता, र राम्रो अभ्यासहरू बुझ्नु प्रभावकारी व्यावसायिक टिम कामको लागि आवश्यक छ, pull requests र code review बुझ्नु सहयोगी विकासको लागि मूल्यवान, बारम्बार-सम्बन्धित ज्ञान हो — व्यावसायिक सफ्टवेयर अभ्यासको एक मूल भाग जसले कोड गुणस्तर, ज्ञान साझेदारी, र टिम प्रभावकारितालाई सीधै प्रभावित गर्दछ, र सफ्टवेयर टिमहरूमा काम गर्ने लगभग सर्वव्यापी पक्ष।