Skip to content

Commit

Permalink
ci: added missing condition for non-cron workflows (#1720)
Browse files Browse the repository at this point in the history
* ci: added missing condition for non-cron workflows

* ci: added slack notification
  • Loading branch information
karl-cardenas-coding committed Oct 27, 2023
1 parent 898537f commit 351e722
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/pull_request.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,7 @@ jobs:
build:
name: Build
needs: [run-ci]
runs-on:
group: 'Doc Runners'
labels: docbot
runs-on: ubuntu-latest
if: ${{ !github.event.pull_request.draft }}
steps:
- name: Checkout Repository
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,9 @@ concurrency:
jobs:
# This is the default workflow that uses the self-hosted runner. The self-hosted runner is slower than the GitHub-hosted large runner, but it does not incurr additional charges.
build-self-hosted-runner:
if: ${{ github.event.inputs.useGitHubHostedLargeRunner != 'true' || github.event_name == 'schedule' }}
name: Build Website
runs-on:
group: 'Doc Runners'
labels: docbot
runs-on: ["self-hosted", "linux", "x64", "vcenter3"]
steps:
- name: Checkout Repository
uses: actions/checkout@v4
Expand Down Expand Up @@ -67,6 +66,7 @@ jobs:
# Use this workflow if you want to use the GitHub-hosted large runner. Useful for scenarios when you need a change to deploy faster than the self-hosted runner can build it.
build-large-runner:
if: ${{ github.event.inputs.useGitHubHostedLargeRunner == 'true' && github.event_name != 'schedule' }}
name: Build Website
runs-on:
group: 'Doc Runners'
Expand Down

0 comments on commit 351e722

Please sign in to comment.