Skip to content

Commit a6bedf2

Browse files
authored
ci: skip coverage from forked PR (#180)
1 parent cf05936 commit a6bedf2

File tree

2 files changed

+8
-9
lines changed

2 files changed

+8
-9
lines changed

.github/workflows/test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ jobs:
5252
thresholds: "60 80"
5353
- name: Add Coverage PR Comment
5454
uses: marocchino/sticky-pull-request-comment@v2
55-
if: ${{ github.event_name == 'pull_request' && github.repository == 'hasura/go-graphql-client' }}
55+
if: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository }}
5656
with:
5757
path: code-coverage-results.md
5858
- name: Dump docker logs on failure

subscription_graphql_ws_test.go

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -576,14 +576,13 @@ func waitForConnectionState(t *testing.T, sc *SubscriptionClient, tickerDuration
576576
defer ticker.Stop()
577577

578578
for {
579-
select {
580-
case <-ticker.C:
581-
if checkFn() {
582-
return true
583-
}
584-
if time.Now().After(deadline) {
585-
return false
586-
}
579+
<-ticker.C
580+
if checkFn() {
581+
return true
582+
}
583+
584+
if time.Now().After(deadline) {
585+
return false
587586
}
588587
}
589588
}

0 commit comments

Comments
 (0)