Skip to content

Commit f0557e7

Browse files
mtfishmanclaude
andauthored
Remove comment-posting from fork-PR skip, use warning annotation (#49)
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 8b35454 commit f0557e7

1 file changed

Lines changed: 2 additions & 26 deletions

File tree

.github/workflows/IntegrationTest.yml

Lines changed: 2 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -50,38 +50,14 @@ jobs:
5050
is_fork="${{ inputs.is-fork-pr }}"
5151
if [[ "$is_fork" == "true" ]] && { [[ "$pkg" == https://* ]] || [[ "$pkg" == git@* ]]; }; then
5252
echo "should-skip=true" >> $GITHUB_OUTPUT
53-
# Derive a comment tag from the full URL for deduplication
54-
tag="integration-skip-$(echo "$pkg" | sed 's|[^a-zA-Z0-9]|-|g')"
55-
echo "comment-tag=$tag" >> $GITHUB_OUTPUT
5653
else
5754
echo "should-skip=false" >> $GITHUB_OUTPUT
5855
fi
5956
60-
- name: "Find existing skip notification comment"
61-
if: steps.skip-check.outputs.should-skip == 'true'
62-
id: find-comment
63-
uses: peter-evans/find-comment@v3
64-
with:
65-
issue-number: ${{ github.event.pull_request.number }}
66-
body-includes: "<!-- ${{ steps.skip-check.outputs.comment-tag }} -->"
67-
68-
- name: "Post skip notification comment"
69-
if: steps.skip-check.outputs.should-skip == 'true'
70-
uses: peter-evans/create-or-update-comment@v4
71-
with:
72-
comment-id: ${{ steps.find-comment.outputs.comment-id }}
73-
issue-number: ${{ github.event.pull_request.number }}
74-
body: |
75-
<!-- ${{ steps.skip-check.outputs.comment-tag }} -->
76-
Private downstream test for `${{ inputs.pkg }}` was skipped (PR was made from a fork). A maintainer can run:
77-
```
78-
/integrationtest ${{ inputs.pkg }}
79-
```
80-
edit-mode: replace
81-
8257
- name: "Private downstream test skipped (fork PR)"
8358
if: steps.skip-check.outputs.should-skip == 'true'
84-
run: echo "Skipped — PR was made from a fork. Use /integrationtest to trigger manually."
59+
run: |
60+
echo "::warning::Private downstream test for ${{ inputs.pkg }} was skipped (PR was made from a fork). A maintainer can run: /integrationtest ${{ inputs.pkg }}"
8561
8662
- name: "No downstream packages configured"
8763
if: inputs.pkg == '__none__' && steps.skip-check.outputs.should-skip != 'true'

0 commit comments

Comments
 (0)