Skip to content

Commit ff67a41

Browse files
authored
ci: properly build dev-app for automatic deploy (#28047)
The deploy script seems rather legacy and also doesn't ensure that the dist directory is actually present. We can simplify the logic and potentially delete the script in the future.
1 parent 47341ac commit ff67a41

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

.github/workflows/deploy-dev-app-main-push.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,10 @@ jobs:
2424
- name: Setup Bazel RBE
2525
uses: angular/dev-infra/github-actions/bazel/configure-remote@fb30926790c6225d553b91a4818cab2fdde4fb4e
2626

27-
- run: yarn -s deploy-dev-app
27+
- name: Building dev-app
28+
run: |
29+
yarn -s bazel build //src/dev-app:web_package --symlink_prefix=dist/
30+
cp -R dist/bin/src/dev-app/web_package dist/dev-app-web-pkg
2831
2932
- name: Configure Firebase deploy target
3033
run: |
@@ -39,6 +42,7 @@ jobs:
3942
repoToken: ''
4043
firebaseServiceAccount: '${{secrets.FIREBASE_PREVIEW_SERVICE_TOKEN}}'
4144
projectId: '${{env.PREVIEW_PROJECT}}'
45+
entryPoint: './'
4246
channelId: '${{env.PREVIEW_CHANNEL}}'
4347

4448
- name: Result

0 commit comments

Comments
 (0)