Skip to content

ci: optimize jobs with path filtering #3074

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 21 commits into from
Jul 23, 2022
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixup! add needs
  • Loading branch information
ammario committed Jul 23, 2022
commit df092330e5034a3117779b9e13cd5ea7fbc0c85f
18 changes: 9 additions & 9 deletions .github/workflows/coder.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ jobs:
timeout-minutes: 5
runs-on: ubuntu-latest
needs: changes
if: needs.changes.outputs.docs-only == 'false'
if: needs.changes.outputs.docs-only == false
steps:
- uses: actions/checkout@v3
- name: Run ShellCheck
Expand All @@ -91,7 +91,7 @@ jobs:
name: "style/lint/typescript"
timeout-minutes: 5
needs: changes
if: needs.changes.outputs.docs-only == 'false'
if: needs.changes.outputs.docs-only == false
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand Down Expand Up @@ -120,7 +120,7 @@ jobs:
timeout-minutes: 5
runs-on: ubuntu-latest
needs: changes
if: needs.changes.outputs.docs-only == 'false'
if: needs.changes.outputs.docs-only == false
steps:
- uses: actions/checkout@v3

Expand Down Expand Up @@ -157,7 +157,7 @@ jobs:
style-fmt:
name: "style/fmt"
needs: changes
if: needs.changes.outputs.docs-only == 'false'
if: needs.changes.outputs.docs-only == false
runs-on: ubuntu-latest
timeout-minutes: 5
steps:
Expand Down Expand Up @@ -191,7 +191,7 @@ jobs:
test-go:
name: "test/go"
needs: changes
if: needs.changes.outputs.docs-only == 'false'
if: needs.changes.outputs.docs-only == false
runs-on: ${{ matrix.os }}
timeout-minutes: 20
strategy:
Expand Down Expand Up @@ -269,7 +269,7 @@ jobs:
test-go-postgres:
name: "test/go/postgres"
needs: changes
if: needs.changes.outputs.docs-only == 'false'
if: needs.changes.outputs.docs-only == false
runs-on: ubuntu-latest
# This timeout must be greater than the timeout set by `go test` in
# `make test-postgres` to ensure we receive a trace of running
Expand Down Expand Up @@ -344,7 +344,7 @@ jobs:
needs: changes
if: |
github.ref == 'refs/heads/main' && !github.event.pull_request.head.repo.fork
&& needs.changes.outputs.docs-only == 'false'
&& needs.changes.outputs.docs-only == false
permissions:
contents: read
id-token: write
Expand Down Expand Up @@ -451,7 +451,7 @@ jobs:
test-js:
name: "test/js"
needs: changes
if: needs.changes.outputs.docs-only == 'false'
if: needs.changes.outputs.docs-only == false
runs-on: ubuntu-latest
timeout-minutes: 20
steps:
Expand Down Expand Up @@ -506,7 +506,7 @@ jobs:
test-e2e:
name: "test/e2e/${{ matrix.os }}"
needs: changes
if: needs.changes.outputs.docs-only == 'false'
if: needs.changes.outputs.docs-only == false
runs-on: ${{ matrix.os }}
timeout-minutes: 20
strategy:
Expand Down