Skip to content

Commit 076b252

Browse files
committed
ci: revert skips of required checks
These were putting certain PRs in an unmergeable state.
1 parent b9936d2 commit 076b252

File tree

7 files changed

+1
-182
lines changed

7 files changed

+1
-182
lines changed

.github/workflows/coder.yaml

+1-42
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ jobs:
3434
runs-on: ubuntu-latest
3535
outputs:
3636
docs-only: ${{ steps.filter.outputs.docs_count == steps.filter.outputs.all_count }}
37-
go: ${{ steps.filter.outputs.go }}
3837
sh: ${{ steps.filter.outputs.sh }}
3938
steps:
4039
- uses: actions/checkout@v3
@@ -49,10 +48,6 @@ jobs:
4948
- 'docs/**'
5049
# For testing:
5150
# - '.github/**'
52-
go:
53-
- "**.go"
54-
- "**.mod"
55-
- "**.sum"
5651
sh:
5752
- "**.sh"
5853
- id: debug
@@ -63,36 +58,16 @@ jobs:
6358
debug-inputs:
6459
needs:
6560
- changes
66-
- pr-context
6761
runs-on: ubuntu-latest
6862
steps:
6963
- id: log
7064
run: |
7165
echo "${{ toJSON(needs) }}"
72-
echo "${{ contains(needs.pr-context.outputs.skips, 'test/go/postgres') }} "
73-
echo "${{ contains(needs.pr-context.outputs.skips, 'test/e2e') }} "
74-
75-
pr-context:
76-
runs-on: ubuntu-latest
77-
outputs:
78-
skips: ${{ steps.pr-context.outputs.skips }}
79-
steps:
80-
- uses: actions/checkout@v3
81-
- uses: actions/setup-go@v3
82-
with:
83-
go-version: "~1.18"
84-
- id: pr-context
85-
env:
86-
GITHUB_CONTEXT: ${{ toJSON(github) }}
87-
run: |
88-
(cd .github/workflows/prcontext && go build)
89-
./.github/workflows/prcontext/prcontext
9066
9167
style-lint-golangci:
9268
name: style/lint/golangci
9369
timeout-minutes: 5
9470
needs: changes
95-
if: needs.changes.outputs.go == 'true'
9671
runs-on: ubuntu-latest
9772
steps:
9873
- uses: actions/checkout@v3
@@ -109,7 +84,6 @@ jobs:
10984
timeout-minutes: 5
11085
runs-on: ubuntu-latest
11186
needs: changes
112-
if: needs.changes.outputs.sh == 'true'
11387
steps:
11488
- uses: actions/checkout@v3
11589
- name: Run ShellCheck
@@ -122,8 +96,6 @@ jobs:
12296
style-lint-typescript:
12397
name: "style/lint/typescript"
12498
timeout-minutes: 5
125-
needs: changes
126-
if: needs.changes.outputs.docs-only == 'false'
12799
runs-on: ubuntu-latest
128100
steps:
129101
- name: Checkout
@@ -207,8 +179,6 @@ jobs:
207179

208180
style-fmt:
209181
name: "style/fmt"
210-
needs: changes
211-
if: needs.changes.outputs.docs-only == 'false'
212182
runs-on: ubuntu-latest
213183
timeout-minutes: 5
214184
steps:
@@ -241,8 +211,6 @@ jobs:
241211
242212
test-go:
243213
name: "test/go"
244-
needs: changes
245-
if: needs.changes.outputs.docs-only == 'false'
246214
runs-on: ${{ matrix.os }}
247215
timeout-minutes: 20
248216
strategy:
@@ -328,12 +296,6 @@ jobs:
328296

329297
test-go-postgres:
330298
name: "test/go/postgres"
331-
needs:
332-
- changes
333-
- pr-context
334-
if: >
335-
needs.changes.outputs.docs-only == 'false' &&
336-
!contains(needs.pr-context.outputs.skips, 'test/go/postgres')
337299
runs-on: ubuntu-latest
338300
# This timeout must be greater than the timeout set by `go test` in
339301
# `make test-postgres` to ensure we receive a trace of running
@@ -514,8 +476,6 @@ jobs:
514476

515477
test-js:
516478
name: "test/js"
517-
needs: changes
518-
if: needs.changes.outputs.docs-only == 'false'
519479
runs-on: ubuntu-latest
520480
timeout-minutes: 20
521481
steps:
@@ -571,8 +531,7 @@ jobs:
571531
name: "test/e2e/${{ matrix.os }}"
572532
needs:
573533
- changes
574-
- pr-context
575-
if: needs.changes.outputs.docs-only == 'false' && !contains(needs.pr-context.outputs.skips, 'test/e2e')
534+
if: needs.changes.outputs.docs-only == 'false'
576535
runs-on: ${{ matrix.os }}
577536
timeout-minutes: 20
578537
strategy:

.github/workflows/prcontext/README.md

-7
This file was deleted.

.github/workflows/prcontext/go.mod

-12
This file was deleted.

.github/workflows/prcontext/go.sum

-16
This file was deleted.

.github/workflows/prcontext/main.go

-39
This file was deleted.

.github/workflows/prcontext/parse.go

-28
This file was deleted.

.github/workflows/prcontext/parse_test.go

-38
This file was deleted.

0 commit comments

Comments
 (0)