Skip to content

Commit

Permalink
Fix syntax error in workflow yaml (facebookincubator#9451)
Browse files Browse the repository at this point in the history
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 facebookincubator#9420

Pull Request resolved: facebookincubator#9451

Reviewed By: bikramSingh91

Differential Revision: D56022463

Pulled By: kgpai

fbshipit-source-id: 79102cc2856db1e18fc4729ee55d602434861e0b
  • Loading branch information
assignUser authored and yanngyoung committed Apr 12, 2024
1 parent 6a6ccaa commit b126be4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/linux-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down Expand Up @@ -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"
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/scheduled.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit b126be4

Please sign in to comment.