From b126be4d7ae9c9cd624db58b24ce1339aeba8ca3 Mon Sep 17 00:00:00 2001 From: Jacob Wujciak-Jens Date: Thu, 11 Apr 2024 10:55:14 -0700 Subject: [PATCH] Fix syntax error in workflow yaml (#9451) Summary: Within `if:`/`${{}}`only single quotes are allowed, due to the fact that broken workflows don't show up in the check suite it got overlooked in https://github.com/facebookincubator/velox/issues/9420 Pull Request resolved: https://github.com/facebookincubator/velox/pull/9451 Reviewed By: bikramSingh91 Differential Revision: D56022463 Pulled By: kgpai fbshipit-source-id: 79102cc2856db1e18fc4729ee55d602434861e0b --- .github/workflows/linux-build.yml | 4 ++-- .github/workflows/scheduled.yml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/linux-build.yml b/.github/workflows/linux-build.yml index 77b8d12f43f2..293a3d3e861b 100644 --- a/.github/workflows/linux-build.yml +++ b/.github/workflows/linux-build.yml @@ -50,7 +50,7 @@ jobs: adapters: name: Linux release with adapters # prevent errors when forks ff their main branch - if: ${{ github.repository == "facebookincubator/velox" }} + if: ${{ github.repository == 'facebookincubator/velox' }} runs-on: 8-core container: ghcr.io/facebookincubator/velox-dev:adapters defaults: @@ -115,7 +115,7 @@ jobs: ubuntu-debug: runs-on: 8-core # prevent errors when forks ff their main branch - if: ${{ github.repository == "facebookincubator/velox" }} + if: ${{ github.repository == 'facebookincubator/velox' }} name: "Ubuntu debug with resolve_dependency" env: CCACHE_DIR: "${{ github.workspace }}/.ccache" diff --git a/.github/workflows/scheduled.yml b/.github/workflows/scheduled.yml index a031526d22d8..4eb90c19d0ae 100644 --- a/.github/workflows/scheduled.yml +++ b/.github/workflows/scheduled.yml @@ -84,7 +84,7 @@ jobs: compile: name: Build # prevent errors when forks ff their main branch - if: ${{ github.repository == "facebookincubator/velox" }} + if: ${{ github.repository == 'facebookincubator/velox' }} runs-on: 16-core container: ghcr.io/facebookincubator/velox-dev:centos8 timeout-minutes: 120