Jose/ansible_role_proxmox_provision
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

style 💎: Remove unnecessary blank line and ensure consistent formatting #25

Merged
Jose merged 4 commits from dev into main 2026-02-14 12:09:29 +01:00
Conversation 1 Commits 4 Files Changed 3 +1 -1
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit af107ca1ee - Show all commits

2
.gitea/workflows/pr-check.yaml
View File

@@ -180,7 +180,7 @@ jobs:
-d '{"labels":["ci-pass"]}' \
"$API/issues/$PR/labels"
fi
# Reopen PR if closed
PR_STATE=$(curl -s -H "Authorization: token $TOKEN" "$API/pulls/$PR" | jq -r '.state')
if [ "$PR_STATE" = "closed" ]; then
gitea-actions commented 2026-02-14 12:08:07 +01:00
Review
Copy Link

[Lines 184-186] [Score: 2] The code reopens the pull request if it is closed. This might lead to unnecessary confusion and potential merge conflicts if the pull request was closed appropriately. It's generally better to have explicit human intervention before reopening closed pull requests.

[Lines 184-186] [Score: 2] The code reopens the pull request if it is closed. This might lead to unnecessary confusion and potential merge conflicts if the pull request was closed appropriately. It's generally better to have explicit human intervention before reopening closed pull requests.