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 +9 -1
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit c5e6c2a431 - Show all commits

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

@@ -104,6 +104,14 @@ jobs:
"$API/issues/$PR/comments"
fi
# Remove ci-pass +label if exists
LABELS=$(curl -s -H "Authorization: token $TOKEN" "$API/issues/$PR/labels" | jq -r '.[] | .name')
if echo "$LABELS" | grep -q "^ci-pass$"; then
curl -s -X DELETE \
-H "Authorization: token $TOKEN" \
"$API/issues/$PR/labels/ci-pass"
fi
# Add label if missing
LABELS=$(curl -s -H "Authorization: token $TOKEN" "$API/issues/$PR/labels" | jq -r '.[] | .name')
if ! echo "$LABELS" | grep -q "^ci-failed$"; then
gitea-actions commented 2026-02-14 12:08:07 +01:00
Review
Copy Link

[Lines 107-117] [Score: 2] The code has been modified to remove the 'ci-pass' label if it exists and add it back if it doesn't. This might lead to inconsistency in label usage, as removing the label could happen before it is added back in certain scenarios.

[Lines 107-117] [Score: 2] The code has been modified to remove the 'ci-pass' label if it exists and add it back if it doesn't. This might lead to inconsistency in label usage, as removing the label could happen before it is added back in certain scenarios.
@@ -155,7 +163,7 @@ jobs:
"$API/issues/$PR/comments"
fi
# Remove label if exists
# Remove ci-failed +label if exists
LABELS=$(curl -s -H "Authorization: token $TOKEN" "$API/issues/$PR/labels" | jq -r '.[] | .name')
if echo "$LABELS" | grep -q "^ci-failed$"; then
curl -s -X DELETE \