From d825bd29c24dd3a5522453b4b837aa024c12e53a Mon Sep 17 00:00:00 2001 From: Pascal Hartig Date: Thu, 13 Jan 2022 04:27:26 -0800 Subject: [PATCH] Fix Windows release job (#3282) Summary: I'm not entirely sure why this broke. Some Powershell versions support `&&` but others don't. Something downgraded us, apparently. Failing job: https://github.com/facebook/flipper/runs/4791210127?check_suite_focus=true Issue: https://github.com/facebook/flipper/issues/3281 Stackoverflow post I copy-pasted from: https://stackoverflow.com/questions/65627536/the-token-is-not-a-valid-statement-separator-in-this-version Pull Request resolved: https://github.com/facebook/flipper/pull/3282 Test Plan: New release, I reckon. Reviewed By: aigoncharov Differential Revision: D33565457 Pulled By: passy fbshipit-source-id: 1751a84fc86d10ee87c60feecb07947e9d5780cf --- .github/workflows/release.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 6919b2bd9..d9dc437f3 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -126,13 +126,13 @@ jobs: with: timeout_minutes: 10 max_attempts: 3 - command: cd ${{env.desktop-directory}} && yarn + command: (cd ${{env.desktop-directory}}) -and (yarn) - name: Build uses: nick-invision/retry@v2.6.0 with: timeout_minutes: 30 max_attempts: 3 - command: cd ${{env.desktop-directory}} && yarn build --win + command: (cd ${{env.desktop-directory}}) -and (yarn build --win) - name: Upload Windows uses: actions/upload-artifact@v1 with: