gitpoint / gitpoint/git-point

Discussion: Internal - Pull Requests management

Đang mở
#625 9 bình luận 0 reaction 0 người được giao Xem trên GitHub
discussion
Ngôn ngữ chính
JavaScript
Star
4.8k
Fork
771
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

This is an attempt to lay out some ground rules in order to improve our handling of PRs.

Hopefully, this will avoid finding ourselves in the current situation introduced by styled-components PRs.

I believe this should be part of some guidelines documentation once discussed & approved.

### Rules for merging PRs

#### Base rules
* Every person who merges a PR should have tested it by itself on at least one device.
* Every merged PR should have a related issue (merger can create one if needed).
* Never merge your own PRs.

#### Exceptions

* Doc: a PR that only changes documentation files (contributors, readme, ..)
* Typo: a PR that fix a typo in translations or in some variable naming
* QuickFix: a PR that fixes an obvious mistake in js logic causing a know issue (bad if condition, ..)
* Crash: a one-linish PR that fix a crash in master.

Additionally, for those cases, if the person submitting the PR is a maintainer, he can commit directly to master (or merge his own PR without approval).

#### Tests

At some point, we will need to start requiring tests for every significant change.

We still need to have more diversified tests samples available in order to ease the task on the pull requester.

#### Enforcements

* UI: If the PR introduces a new UI, or a change in the UI (styled-components, ..), the PR **should** be tested on both iOS & Android by the person who merges it.
* UI: The PR needs to have screenshots for both Android & iOS. (either the pull-requester or merger should provide them in the discussion)
* i18n: The PR have to be *approved* by at least one native speaker before merge can happen

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Bắt đầu bằng việc đọc phần thảo luận của issue và các quy tắc được đề xuất trong đó để merge pull requests, bao gồm các yêu cầu về testing, UI, screenshot và i18n. Không có điểm vào nào cho file, test hoặc tài liệu được nêu; để hoàn thành cần đạt được sự thống nhất về các hướng dẫn và xác định rõ vị trí của tài liệu.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
android, ios, javascript
Lĩnh vực
documentation
Loại issue
Tài liệu
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Cần làm rõ
Mức phù hợp với người mới
25/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.