github-tools / github-tools/github
Issues with protected branches: Github is confusing Pull Requests review and push protection
- Ngôn ngữ chính
- JavaScript
- Star
- 3.8k
- Fork
- 809
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
As a Github user, i want to request review for Pull Requests, but, i want to able push on branch.
Setup:
1) Create a branch
2) Protect it, asking review for Pull Requests
3) Do not protect push
Expected Behaviour:
1) Pull Requests require review
2) Push is open on following branch
Actual Scenario:
1) Pull Requests is requiring reviews
2) Got following error while pushing:
```
ramon@Ramons-Mac-mini:~/GIT/SAT-ROBOT (master)$ git push robot master
Counting objects: 2, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (2/2), done.
Writing objects: 100% (2/2), 295 bytes | 0 bytes/s, done.
Total 2 (delta 1), reused 0 (delta 0)
remote: Resolving deltas: 100% (1/1), completed with 1 local object.
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: At least one approved review is required by reviewers with write access.
To https://github.com/SymphonyOSF/SAT-Robot.git
! [remote rejected] master -> master (protected branch hook declined)
error: failed to push some refs to 'https://github.com/SymphonyOSF/SAT-Robot.git'
```
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Hướng nghiên cứu
Start by reproducing the listed protected-branch setup and compare the expected behavior with the GH006 push rejection. The issue mentions no repository file, test, or entry point, so first determine whether the behavior belongs in this GitHub API wrapper or must be clarified as an external GitHub behavior. Done would require a confirmed scope and an actionable change or resolution.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- github
- Lĩnh vực
- devtools
- Loại issue
- Lỗi
- Độ 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