Skip to content

Commit

Permalink
Change mobile binary size comparison workflow to use release build (e…
Browse files Browse the repository at this point in the history
…nvoyproxy#25189)

Signed-off-by: caschoener <schoener@google.com>
  • Loading branch information
caschoener authored and VishalDamgude committed Feb 2, 2023
1 parent d3a9fc3 commit f1fce5a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/perf.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
run: |
cd mobile && ./bazelw build \
--config=sizeopt \
--config=remote-ci-linux-clang \
--config=release-common \
--remote_header="Authorization=Bearer $GITHUB_TOKEN" \
//test/performance:test_binary_size
- uses: actions/upload-artifact@v3
Expand Down Expand Up @@ -58,7 +58,7 @@ jobs:
git checkout main && git pull origin main
cd mobile && ./bazelw build \
--config=sizeopt \
--config=remote-ci-linux-clang \
--config=release-common \
--remote_header="Authorization=Bearer $GITHUB_TOKEN" \
//test/performance:test_binary_size
- uses: actions/upload-artifact@v3
Expand Down Expand Up @@ -90,4 +90,4 @@ jobs:
zip -9 dist/main.zip dist/main.stripped
zip -9 dist/current.zip dist/current.stripped
- name: 'Test size regression'
run: cd mobile && ./ci/test_size_regression.sh dist/main.zip dist/current.zip
run: cd mobile && ./ci/test_size_regression.sh ../dist/main.zip ../dist/current.zip

0 comments on commit f1fce5a

Please sign in to comment.