Skip to content

Commit

Permalink
try building the theme in the initial build step so it only has to ha…
Browse files Browse the repository at this point in the history
…ppen once

Signed-off-by: Chris Abraham <[email protected]>
  • Loading branch information
cjyabraham committed May 22, 2024
1 parent 01598b2 commit 16f52e8
Showing 1 changed file with 11 additions and 17 deletions.
28 changes: 11 additions & 17 deletions .github/workflows/build_test_and_deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ jobs:
export GITHUB_WORKFLOW_URL=https://github.com/$GITHUB_REPOSITORY/actions/runs/$GITHUB_RUN_ID
./.github/workflows/set-environment
build_php:
build_site:
container:
image: quay.io/pantheon-public/build-tools-ci:6.x
options: --user root
Expand Down Expand Up @@ -122,6 +122,14 @@ jobs:
- name: run PHP build step
run: composer -n build-assets

- uses: actions/setup-node@v4
with:
node-version: '14.21.3'

- name: Compile css/js
run: cd ./web/wp-content/themes/lfevents/; npm install; npm run build


code_sniff_unit_test:
runs-on: ubuntu-latest
container:
Expand All @@ -143,7 +151,7 @@ jobs:
image: quay.io/pantheon-public/build-tools-ci:6.x
options: --user root
runs-on: ubuntu-latest
needs: [configure_env_vars, build_php]
needs: [configure_env_vars, build_site]
steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down Expand Up @@ -189,13 +197,6 @@ jobs:
- name: sync build artifact
run: rsync -rz ./web ./vendor .

- uses: actions/setup-node@v4
with:
node-version: '14.21.3'

- name: Compile css/js
run: cd ./web/wp-content/themes/lfevents/; npm install; npm run build

# Deploy to Pantheon
- name: deploy to Pantheon
run: |
Expand All @@ -211,7 +212,7 @@ jobs:
image: quay.io/pantheon-public/build-tools-ci:6.x
options: --user root
runs-on: ubuntu-latest
needs: [configure_env_vars, build_php]
needs: [configure_env_vars, build_site]
steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down Expand Up @@ -257,13 +258,6 @@ jobs:
- name: sync build artifact
run: rsync -rz ./web ./vendor .

- uses: actions/setup-node@v4
with:
node-version: '14.21.3'

- name: Compile css/js
run: cd ./web/wp-content/themes/lfevents/; npm install; npm run build

# Deploy to Pantheon
- name: deploy to Pantheon
run: |
Expand Down

0 comments on commit 16f52e8

Please sign in to comment.