Skip to content

Commit 28446ee

Browse files
committed
fix(cI): Install coreutils in mac github workers for smoke test (#3563)
Signed-off-by: Achal Shah <achals@gmail.com>
1 parent ab06fa9 commit 28446ee

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

.github/workflows/build_wheels.yml

+4-1
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ jobs:
5252
echo $HIGHEST_SEMVER_TAG
5353
5454
build-python-wheel:
55-
name: Build wheels on ${{ matrix.os }}
55+
name: Build wheels
5656
runs-on: ubuntu-latest
5757
steps:
5858
- uses: actions/checkout@v2
@@ -164,6 +164,9 @@ jobs:
164164
with:
165165
name: wheels
166166
path: dist
167+
- name: Install OS X dependencies
168+
if: matrix.os == 'macos-10.15'
169+
run: brew install coreutils
167170
- name: Install wheel
168171
if: ${{ !matrix.from-source }}
169172
# try to install all wheels; only the current platform wheel should be actually installed

.github/workflows/release.yml

+1-2
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ jobs:
9292

9393

9494
publish-web-ui-npm:
95-
if: github.repository == 'feast-dev/feast'
95+
if: github.repository == 'feast-dev/feast' and github.event.inputs.publish_ui == 'true'
9696
needs: [validate_version_bumps, get_dry_release_versions]
9797
runs-on: ubuntu-latest
9898
env:
@@ -115,7 +115,6 @@ jobs:
115115
working-directory: ./ui
116116
run: yarn build:lib
117117
- name: Publish UI package
118-
if: github.event.inputs.publish_ui == 'true'
119118
working-directory: ./ui
120119
run: npm publish
121120
env:

0 commit comments

Comments
 (0)