forked from PelicanPlatform/pelican
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add GHA that enforces labeling and linking in PRs
- Loading branch information
1 parent
8e5f60a
commit 37b41e2
Showing
1 changed file
with
55 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,55 @@ | ||
name: PR and Issue Validation | ||
|
||
on: | ||
pull_request: | ||
# one limitation here is that there's no trigger to re-run any time we "connect" or "disconnect" an issue | ||
types: [opened, edited, labeled, unlabeled, synchronize] | ||
|
||
jobs: | ||
validate-pr: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Check out the repository | ||
uses: actions/checkout@v2 | ||
|
||
- name: Validate PR has labels | ||
id: check_labels | ||
run: | | ||
PR_LABELS=$(jq -r '.pull_request.labels | length' $GITHUB_EVENT_PATH) | ||
if [ "$PR_LABELS" -eq "0" ]; then | ||
echo "No labels found on the pull request." | ||
exit 1 | ||
fi | ||
- name: Validate PR is linked to an issue | ||
id: check_linked_issues | ||
run: | | ||
PR_NUMBER=$(jq -r '.pull_request.number' $GITHUB_EVENT_PATH) | ||
REPO_OWNER=$(jq -r '.repository.owner.login' $GITHUB_EVENT_PATH) | ||
REPO_NAME=$(jq -r '.repository.name' $GITHUB_EVENT_PATH) | ||
TIMELINE_JSON=$(curl -s "https://api.github.com/repos/$REPO_OWNER/$REPO_NAME/issues/$PR_NUMBER/timeline") | ||
# Count the number of times the timeline sees a "connected" event and subract the number of "disconnected" events | ||
# We might also consider using the "cross-referenced" event in the future if actual connecting/disconnecting is too heavy-handed | ||
LINKED_ISSUES=$(echo "$TIMELINE_JSON" | jq ' | ||
reduce .[] as $event ( | ||
0; | ||
if $event.event == "connected" then | ||
. + 1 | ||
elif $event.event == "disconnected" then | ||
. - 1 | ||
else | ||
. | ||
end | ||
)') | ||
# If the sum is 0, then no linked issues were found | ||
if [ "$LINKED_ISSUES" -eq "0" ]; then | ||
echo "❌ No linked issues found in the pull request." | ||
exit 1 | ||
elif [ "$LINKED_ISSUES" -lt "0" ]; then | ||
echo "Error: More disconnected events than connected events. This shouldn't be possible and likely indicates a big ol' 🪲" | ||
exit 1 | ||
else | ||
echo "Linked issues found: $LINKED_ISSUES" | ||
fi |