mirror of
https://github.com/community-scripts/ProxmoxVE
synced 2025-01-10 19:05:09 +00:00
Merge pull request #56 from havardthom/remove-action
Remove unnecessary code
This commit is contained in:
commit
10a7b17e83
28
.github/pr-review.yml
vendored
28
.github/pr-review.yml
vendored
@ -1,28 +0,0 @@
|
||||
name: Pull Request Review Enforcement
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
types: [opened, reopened, synchronize]
|
||||
|
||||
jobs:
|
||||
enforce-review-requirements:
|
||||
runs-on: ubuntu-latest
|
||||
|
||||
steps:
|
||||
- name: Check out the code
|
||||
uses: actions/checkout@v2
|
||||
|
||||
- name: Verify Reviewer Approvals
|
||||
id: review-check
|
||||
run: |
|
||||
reviews=$(gh api repos/$GITHUB_REPOSITORY/pulls/$GITHUB_EVENT_NUMBER/reviews --jq '[.[] | select(.state == "APPROVED") | .user.login] | unique | length')
|
||||
if [ "$reviews" -lt 2 ]; then
|
||||
echo "PR requires at least 2 approvals from the Contributor team."
|
||||
exit 1
|
||||
fi
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
|
||||
- name: Confirm Ready to Merge
|
||||
if: ${{ steps.review-check.outputs.reviews == '2' }}
|
||||
run: echo "PR is ready to be merged."
|
5
.github/pull_request_template.md
vendored
5
.github/pull_request_template.md
vendored
@ -27,8 +27,3 @@ If there are other pull requests or discussions related to this change, please l
|
||||
- Related PR #1
|
||||
- Related PR #2
|
||||
|
||||
## Review Guidelines
|
||||
|
||||
- [ ] This PR requires review and approval by at least **two contributors**.
|
||||
- [ ] If changes affect critical functionality, please notify the project maintainers.
|
||||
- [ ] For major updates, ensure all conversations on this PR are resolved before merging.
|
||||
|
Loading…
Reference in New Issue
Block a user