|
| 1 | +--- |
| 2 | +title: بازبینی برای تأییدکنندگان و بازبینها |
| 3 | +linktitle: برای تأییدکنندگان و بازبینها |
| 4 | +slug: for-approvers |
| 5 | +content_type: concept |
| 6 | +weight: 20 |
| 7 | +--- |
| 8 | + |
| 9 | +<!-- overview --> |
| 10 | + |
| 11 | +بازبینها ([Reviewers](/docs/contribute/participate/#reviewers)) و |
| 12 | +تأییدکنندگان ([Approvers](/docs/contribute/participate/#approvers)) در SIG Docs |
| 13 | +هنگام بازبینی یک تغییر چند کار اضافه نیز انجام میدهند. |
| 14 | + |
| 15 | +هر هفته یکی از تأییدکنندگان مستندات (docs approver) داوطلب میشود تا Pull Requestها را تریاژ (triage) و بازبینی کند. |
| 16 | +این فرد در آن هفته "PR Wrangler" است. برای اطلاعات بیشتر، به |
| 17 | +[زمانبندی PR Wrangler](https://github.com/kubernetes/website/wiki/PR-Wranglers) مراجعه کنید. |
| 18 | +برای تبدیل شدن به PR Wrangler، در نشست هفتگی SIG Docs شرکت کنید و داوطلب شوید. |
| 19 | +حتی اگر در برنامهٔ همین هفته نام شما نیست، هنوز هم میتوانید Pull Requestهایی (PRها) را که در حال حاضر تحت بازبینی فعال نیستند، بررسی کنید. |
| 20 | + |
| 21 | +علاوه بر چرخشی بودن، یک بات بازبینها و تأییدکنندگان PR را بر اساس مالکیت فایل (پرونده)های تحت تأثیر تعیین میکند. |
| 22 | + |
| 23 | +<!-- body --> |
| 24 | + |
| 25 | +## بازبینی یک PR |
| 26 | + |
| 27 | +مستندات کوبرنتیز از |
| 28 | +[فرآیند بازبینی کد کوبرنتیز](https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md#the-code-review-process) |
| 29 | +پیروی میکند. |
| 30 | + |
| 31 | +هر آنچه در [بازبینی یک Pull Request](/docs/contribute/review/reviewing-prs) آمده است صدق میکند، |
| 32 | +اما بازبینها و تأییدکنندگان باید کارهای زیر را نیز انجام دهند: |
| 33 | + |
| 34 | +- استفاده از دستور Prow `/assign` برای اختصاص دادن یک بازبین مشخص به یک Pull Request در صورت نیاز. این کار بهویژه زمانی اهمیت بیشتری دارد که نیاز به درخواست بازبینی فنی از مشارکتکنندگان کد باشد. |
| 35 | + |
| 36 | + {{< note >}} |
| 37 | + به فیلد `reviewers` در front-matter بالای فایل (پرونده) Markdown نگاه کنید تا ببینید چه کسی میتواند بازبینی فنی انجام دهد. |
| 38 | + {{< /note >}} |
| 39 | + |
| 40 | +- اطمینان از اینکه PR مطابق با [راهنمای محتوا](/docs/contribute/style/content-guide/) و [راهنمای سبک](/docs/contribute/style/style-guide/) است؛ |
| 41 | + اگر اینطور نیست، نویسنده را به بخش مربوطه در راهنما(ها) ارجاع دهید. |
| 42 | +- استفاده از گزینه **Request Changes** در GitHub در صورت لزوم برای پیشنهاد تغییر به نویسنده PR. |
| 43 | +- تغییر وضعیت بازبینی خود در GitHub با استفاده از دستورات Prow `/approve` یا `/lgtm` |
| 44 | + اگر پیشنهادهای شما اعمال شدهاند. |
| 45 | + |
| 46 | +## commit در PR شخص دیگر |
| 47 | + |
| 48 | +گذاشتن نظر در PR مفید است، اما گاهی ممکن است لازم باشد در Pull Request شخص دیگری commit کنید. |
| 49 | + |
| 50 | +هرگز کار شخص دیگری را «بهدست نگیرید» مگر اینکه خود او صراحتاً درخواست کرده باشد |
| 51 | +یا قصد داشته باشید یک PR قدیمی و رهاشده را احیا کنید. |
| 52 | +گرچه این کار ممکن است در کوتاهمدت سریعتر باشد، اما فرصت مشارکت را از آن فرد میگیرد. |
| 53 | + |
| 54 | +روش کار بستگی دارد به اینکه آیا نیاز دارید فایلی (پروندهای) را ویرایش کنید که در دامنهٔ PR است |
| 55 | +یا فایلی (پروندهای) را که هنوز در آن PR تغییری نکرده است. |
| 56 | + |
| 57 | +شما نمیتوانید در PR شخص دیگر commit کنید اگر یکی از شرایط زیر برقرار باشد: |
| 58 | + |
| 59 | +- اگر نویسنده PR شاخه (branch) خود را مستقیماً به مخزن |
| 60 | + [https://github.com/kubernetes/website/](https://github.com/kubernetes/website/) |
| 61 | + پوش کرده باشد. تنها بازبینیکنندهای با دسترسی push میتواند در PR کاربر دیگر commit کند. |
| 62 | + |
| 63 | + {{< note >}} |
| 64 | + نویسنده را تشویق کنید دفعه بعد شاخه خود را در fork شخصی بسازد و سپس PR باز کند. |
| 65 | + {{< /note >}} |
| 66 | + |
| 67 | +- نویسنده PR بهصراحت ویرایش توسط تأییدکنندگان را غیرفعال کرده باشد. |
| 68 | + |
| 69 | +## دستورات Prow برای بازبینی |
| 70 | + |
| 71 | +[Prow](https://github.com/kubernetes/test-infra/blob/master/prow/README.md) |
| 72 | +سیستم CI/CD مبتنی بر کوبرنتیز است که روی Pull Requestها اجرا میشود. |
| 73 | +Prow امکان استفاده از دستورات شبیه به چتبات را فراهم میکند تا کارهای GitHub مانند [افزودن یا حذف برچسبها](#افزودن-و-حذف-برچسب-های-issue)، بستن Issueها و تعیین تاییدکنندگان را مدیریت کند. |
| 74 | +دستورات Prow بهصورت کامنت در GitHub و با فرمت `/<command-name>` نوشته میشوند. |
| 75 | + |
| 76 | +رایجترین دستورات Prow برای بازبینها و تأییدکنندگان عبارتند از: |
| 77 | + |
| 78 | +{{< table caption="دستورات Prow برای بازبینی" >}} |
| 79 | +دستور Prow | محدودیت نقش | توضیحات |
| 80 | +:-----------|:------------|:---------- |
| 81 | +`/lgtm` | اعضای سازمان | نشان میدهد بازبینی شما تمام شده و از تغییرات راضی هستید. |
| 82 | +`/approve` | تایید کنندگان | PR را برای ادغام تأیید میکند. |
| 83 | +`/assign` | همه | شخصی را برای بازبینی یا تأیید PR تعیین میکند. |
| 84 | +`/close` | اعضای سازمان | یک Issue یا PR را میبندد. |
| 85 | +`/hold` | همه | برچسب `do-not-merge/hold` را اضافه میکند و مانع ادغام خودکار PR میشود. |
| 86 | +`/hold cancel` | همه | برچسب `do-not-merge/hold` را حذف میکند. |
| 87 | +{{< /table >}} |
| 88 | + |
| 89 | +برای دیدن لیست کامل دستورات قابل استفاده در PR، به |
| 90 | +[Prow مرجع دستورات](https://prow.k8s.io/command-help?repo=kubernetes%2Fwebsite) مراجعه کنید. |
| 91 | + |
| 92 | +## دستهبندی و اولویتبندی Issueها |
| 93 | + |
| 94 | +بهطور کلی، SIG Docs از فرآیند |
| 95 | +[تریاژ Issueها در کوبرنتیز](https://github.com/kubernetes/community/blob/master/contributors/guide/issue-triage.md) |
| 96 | +پیروی میکند و از همان برچسبها استفاده میکند. |
| 97 | + |
| 98 | +این [فیلتر](https://github.com/kubernetes/website/issues?q=is%3Aissue+is%3Aopen+-label%3Apriority%2Fbacklog+-label%3Apriority%2Fimportant-longterm+-label%3Apriority%2Fimportant-soon+-label%3Atriage%2Fneeds-information+-label%3Atriage%2Fsupport+sort%3Acreated-asc) |
| 99 | +در GitHub، فهرستی از issueها را پیدا میکند که ممکن است به تریاژ نیاز داشته باشند. |
| 100 | + |
| 101 | +### Triage یک Issue |
| 102 | + |
| 103 | +1. اعتبارسنجی Issue |
| 104 | + |
| 105 | + - مطمئن شوید که Issue مربوط به مستندات وبسایت است. برخی Issueها با پاسخ سریع یا ارجاع بسته میشوند. |
| 106 | + (به بخش [درخواستهای پشتیبانی یا گزارش باگ کد](#درخواست-های-پشتیبانی-یا-گزارش-باگ-کد) مراجعه کنید.) |
| 107 | + - بررسی کنید که Issue ارزش پیگیری دارد یا نه. |
| 108 | + - اگر Issue جزئیات کافی ندارد یا قالب بهدرستی پر نشده، برچسب `triage/needs-information` اضافه کنید. |
| 109 | + - اگر Issue هم `lifecycle/stale` و هم `triage/needs-information` دارد، آن را ببندید. |
| 110 | + |
| 111 | +2. افزودن برچسب اولویت (برای جزئیات کامل به |
| 112 | + [دستورالعملهای تریاژ Issueها](https://github.com/kubernetes/community/blob/master/contributors/guide/issue-triage.md#define-priority) مراجعه کنید) |
| 113 | + |
| 114 | +{{< table caption="برچسبهای Issue" >}} |
| 115 | +برچسب | توضیحات |
| 116 | +:------------|:------------------ |
| 117 | +`priority/critical-urgent` | همین حالا انجام دهید. |
| 118 | +`priority/important-soon` | ظرف ۳ ماه انجام دهید. |
| 119 | +`priority/important-longterm` | ظرف ۶ ماه انجام دهید. |
| 120 | +`priority/backlog` | قابل تعویق نامحدود؛ هر زمان منابع در دسترس بود انجام دهید. |
| 121 | +`priority/awaiting-more-evidence` | نگهدارنده یک Issue بالقوه خوب تا گم نشود. |
| 122 | +`help` یا `good first issue` | مناسب برای کسی با تجربه بسیار کم در کوبرنتیز یا SIG Docs. برای اطلاعات بیشتر، [Help Wanted و Good First Issue](https://kubernetes.dev/docs/guide/help-wanted/) را ببینید. |
| 123 | +{{< /table >}} |
| 124 | + |
| 125 | +در صورت صلاحدید، مالکیت یک Issue را بر عهده بگیرید و برای آن PR ارسال کنید |
| 126 | +(بهویژه اگر سریع انجام میشود یا مرتبط با کاری است که همین حالا انجام میدهید). |
| 127 | + |
| 128 | +اگر درباره تریاژ یک Issue پرسشی داشتید، در کانال `#sig-docs` در Slack یا |
| 129 | +[لیست ایمیل kubernetes-sig-docs](https://groups.google.com/forum/#!forum/kubernetes-sig-docs) بپرسید. |
| 130 | + |
| 131 | +## افزودن و حذف برچسب های Issue |
| 132 | + |
| 133 | +برای افزودن برچسب، یکی از قالبهای زیر را در کامنت بگذارید: |
| 134 | + |
| 135 | +- `/<label-to-add>` (مثال: `/good-first-issue`) |
| 136 | +- `/<label-category> <label-to-add>` (مثال: `/triage needs-information` یا `/language ja`) |
| 137 | + |
| 138 | +برای حذف برچسب: |
| 139 | + |
| 140 | +- `/remove-<label-to-remove>` (مثال: `/remove-help`) |
| 141 | +- `/remove-<label-category> <label-to-remove>` (مثال: `/remove-triage needs-information`) |
| 142 | + |
| 143 | +در هر دو حالت، برچسب باید از قبل وجود داشته باشد. اگر تلاش کنید برچسبی که وجود ندارد را اضافه کنید، فرمان بدون پیام نادیده گرفته میشود. |
| 144 | + |
| 145 | +لیست همه برچسبها در بخش [برچسبهای مخزن وبسایت](https://github.com/kubernetes/website/labels) موجود است. |
| 146 | +همه برچسبها توسط SIG Docs استفاده نمیشوند. |
| 147 | + |
| 148 | +### برچسبهای چرخه عمر Issue |
| 149 | + |
| 150 | +Issueها معمولاً بهسرعت باز و بسته میشوند. بااینحال، گاهی یک Issue پس از باز شدن غیرفعال میشود و گاهی لازم است بیش از ۹۰ روز باز بماند. |
| 151 | + |
| 152 | +{{< table caption="برچسبهای چرخه عمر Issue" >}} |
| 153 | +برچسب | توضیحات |
| 154 | +:------------|:------------------ |
| 155 | +`lifecycle/stale` | پس از ۹۰ روز بدون فعالیت، Issue بهطور خودکار با این برچسب علامتگذاری میشود. اگر چرخه عمر بهصورت دستی با فرمان `/remove-lifecycle stale` برگردانده نشود، Issue خودکار بسته خواهد شد. |
| 156 | +`lifecycle/frozen` | Issue دارای این برچسب پس از ۹۰ روز غیرفعال شدن منقضی نمیشود. کاربر این برچسب را دستی برای Issueهایی اضافه میکند که باید مدتزمان بسیار طولانیتری باز بمانند، مانند آنهایی که برچسب `priority/important-longterm` دارند. |
| 157 | +{{< /table >}} |
| 158 | + |
| 159 | +## مدیریت انواع خاص Issueها |
| 160 | + |
| 161 | +SIG Docs بهاندازهای با انواع زیر از Issue مواجه میشود که لازم است شیوه رسیدگی به آنها مستند شود. |
| 162 | + |
| 163 | +### Issueهای تکراری |
| 164 | + |
| 165 | +اگر یک مشکل واحد بیش از یک Issue باز دارد، آنها را در یک Issue ادغام کنید. |
| 166 | +تصمیم بگیرید کدام Issue باز بماند (یا یک Issue جدید باز کنید)، سپس تمام اطلاعات مرتبط را منتقل کرده و Issueهای مرتبط را پیوند کنید. |
| 167 | +در نهایت، به همه Issueهای دیگر که همان مشکل را توصیف میکنند برچسب `triage/duplicate` بزنید و آنها را ببندید. داشتن یک Issue منفرد، سردرگمی را کاهش میدهد و از دوبارهکاری جلوگیری میکند. |
| 168 | + |
| 169 | +### Issueهای مربوط به پیوندهای خراب |
| 170 | + |
| 171 | +اگر Issue پیوند مرده در مستندات API یا `kubectl` است، تا زمانی که مشکل کاملاً درک شود به آن برچسب `/priority critical-urgent` بدهید. |
| 172 | +به تمام Issueهای پیوند مرده دیگر برچسب `/priority important-longterm` بدهید، چون باید بهصورت دستی رفع شوند. |
| 173 | + |
| 174 | +### Issueهای وبنوشت (blog) |
| 175 | + |
| 176 | +انتظار میرود [مطالب وبنوشت (blog) کوبرنتیز](/blog/) با گذر زمان قدیمی شوند؛ بنابراین فقط مطالب کمتر از یکسال را نگهداری میکنیم. |
| 177 | +اگر Issue مربوط به مطلبی قدیمیتر از یک سال است، معمولاً باید Issue را بدون رفع ببندید. |
| 178 | + |
| 179 | +میتوانید هنگام بستن PR، پیوندی به |
| 180 | +[بهروزرسانی و نگهداری مقاله](/docs/contribute/blog/#maintenance) |
| 181 | +بفرستید. |
| 182 | + |
| 183 | +در صورت وجود توجیه مناسب، ایجاد استثنا اشکالی ندارد. |
| 184 | + |
| 185 | +### درخواست های پشتیبانی یا گزارش باگ کد |
| 186 | + |
| 187 | +برخی از Issueهای مربوط به مستندات در واقع به کد اصلی مربوط میشوند یا درخواست کمک هستند، |
| 188 | +برای مثال زمانی که یک آموزش (tutorial) درست کار نمیکند. |
| 189 | +برای Issueهایی که به مستندات مربوط نیستند، آنها را با برچسب `kind/support` ببندید |
| 190 | +و در یک کامنت، درخواستکننده را به کانالهای پشتیبانی دیگر (مانند Slack یا Stack Overflow) راهنمایی کنید. |
| 191 | +اگر Issue مربوط به باگ در یک قابلیت باشد، درخواستکننده را به مخزن مرتبط هدایت کنید |
| 192 | +(مخزن `kubernetes/kubernetes` نقطهٔ شروع مناسبی است). |
| 193 | + |
| 194 | +نمونه پاسخ به درخواست پشتیبانی: |
| 195 | + |
| 196 | +```none |
| 197 | +This issue sounds more like a request for support and less |
| 198 | +like an issue specifically for docs. I encourage you to bring |
| 199 | +your question to the `#kubernetes-users` channel in |
| 200 | +[Kubernetes slack](https://slack.k8s.io/). You can also search |
| 201 | +resources like |
| 202 | +[Stack Overflow](https://stackoverflow.com/questions/tagged/kubernetes) |
| 203 | +for answers to similar questions. |
| 204 | +
|
| 205 | +You can also open issues for Kubernetes functionality in |
| 206 | +https://github.com/kubernetes/kubernetes. |
| 207 | +
|
| 208 | +If this is a documentation issue, please re-open this issue. |
| 209 | +``` |
| 210 | + |
| 211 | +نمونه پاسخ به گزارش باگ کد: |
| 212 | + |
| 213 | +```none |
| 214 | +This sounds more like an issue with the code than an issue with |
| 215 | +the documentation. Please open an issue at |
| 216 | +https://github.com/kubernetes/kubernetes/issues. |
| 217 | +
|
| 218 | +If this is a documentation issue, please re-open this issue. |
| 219 | +``` |
| 220 | + |
| 221 | +### squash کردن (Squashing) |
| 222 | + |
| 223 | +بهعنوان یک تأییدکننده، هنگام بازبینی Pull Requestها (PRها) ممکن است در شرایط مختلف یکی از کارهای زیر را انجام دهید: |
| 224 | + |
| 225 | +- به مشارکتکننده توصیه کنید commitهایش را squash کند. |
| 226 | +- commitها را بهجای مشارکتکننده squash کنید. |
| 227 | +- به مشارکتکننده توصیه کنید هنوز squash نکند. |
| 228 | +- مانع از squash شدن شوید. |
| 229 | + |
| 230 | +**توصیه به مشارکتکنندگان برای squash**: یک مشارکتکننده تازهوارد ممکن است نداند که باید commitهایش را در PR squash کند. در این صورت، او را راهنمایی کنید، پیوندهای مفید در اختیارش بگذارید و در صورت نیاز پیشنهاد کمک بدهید. چند پیوند مفید: |
| 231 | + |
| 232 | +- [باز کردن Pull Request و squash commitها](/docs/contribute/new-content/open-a-pr#squashing-commits) برای مشارکتکنندگان مستندات |
| 233 | +- [گردش کار GitHub](https://www.k8s.dev/docs/guide/github-workflow/) (شامل نمودارها) برای توسعهدهندگان |
| 234 | + |
| 235 | +**squash commit برای مشارکتکنندگان**: اگر مشارکتکننده ممکن است در squash کردن مشکل داشته باشد یا فشار زمانی برای ادغام PR وجود دارد، میتوانید squash را برای او انجام دهید: |
| 236 | + |
| 237 | +- مخزن `kubernetes/website` |
| 238 | + [برای اجازه squash هنگام ادغام PR پیکربندی شده است](https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/configuring-commit-squashing-for-pull-requests). کافی است دکمه _Squash commits_ را بزنید. |
| 239 | +- در یک PR، اگر مشارکتکننده اجازهٔ مدیریت PR را به نگهدارندگان (maintainers) بدهد، |
| 240 | + میتوانید commitهای او را squash کنید و fork او را با نتیجهٔ نهایی بهروزرسانی نمایید. |
| 241 | + پیش از انجام squash، به او توصیه کنید آخرین تغییرات خود را ذخیره کرده و به PR پوش کند. |
| 242 | + پس از squash نیز به او بگویید commit squash شده را به کلون محلی خود pull کند. |
| 243 | + |
| 244 | +- همچنین میتوانید با استفاده از یک برچسب (label) کاری کنید که GitHub commitها را squash کند، |
| 245 | + بهطوری که Tide / GitHub این کار را انجام دهند؛ |
| 246 | + یا هنگام ادغام PR روی دکمهٔ _Squash commits_ کلیک کنید. |
| 247 | + |
| 248 | +**توصیه برای پرهیز از squash** |
| 249 | + |
| 250 | +- اگر یک commit تغییرات خرابکننده یا نادرستی انجام داده باشد و commit آخر برای بازگرداندن (revert) آن خطا باشد، |
| 251 | + در این حالت commitها را squash نکنید. |
| 252 | + حتی اگر در تب "Files changed" در PR در GitHub و در پیشنمایش Netlify همهچیز درست بهنظر برسد، |
| 253 | + ادغام چنین PRی میتواند برای دیگر مشارکتکنندگان باعث ایجاد conflict در rebase یا merge شود. |
| 254 | + در چنین شرایطی، متناسب با صلاحدید خود مداخله کنید تا از ایجاد مشکل برای دیگر مشارکتکنندگان جلوگیری شود. |
| 255 | + |
| 256 | +**هرگز squash نکنید** |
| 257 | + |
| 258 | +اگر در حال راهاندازی یک بومیسازی (localization) یا انتشار مستندات برای یک نسخهٔ جدید هستید، |
| 259 | +در این حالت شاخهای را ادغام میکنید که از fork یک کاربر نیست؛ |
| 260 | +در چنین شرایطی هرگز commitها را squash نکنید. |
| 261 | +عدم squash در اینجا ضروری است زیرا باید تاریخچهٔ کامل commitها برای آن فایل (پرونده)ها حفظ شود. |
0 commit comments