Skip to content

Commit

Permalink
Merge pull request #7653 from ampproject/dependabot/npm_and_yarn/word…
Browse files Browse the repository at this point in the history
…press-packages-f46137a90d

Bump the wordpress-packages group with 29 updates
  • Loading branch information
westonruter authored Nov 2, 2023
2 parents 1a0c049 + 3d3b154 commit 66b8668
Show file tree
Hide file tree
Showing 3 changed files with 660 additions and 704 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/build-test-measure.yml
Original file line number Diff line number Diff line change
Expand Up @@ -371,13 +371,15 @@ jobs:
uses: actions/checkout@v4

- name: Setup PHP and Composer
if: needs.pre-run.outputs.changed-php-count > 0
uses: ./.github/actions/setup-php-composer
with:
php-version: ${{ matrix.php }}
coverage: ${{ matrix.coverage && 'xdebug' || 'none' }}
tools: 'composer,phpunit:${{ matrix.phpunit }}'

- name: Setup NodeJS and NPM
if: needs.pre-run.outputs.changed-php-count > 0
uses: ./.github/actions/setup-node-npm

- name: Build plugin
Expand Down Expand Up @@ -422,7 +424,7 @@ jobs:
working-directory: ${{ env.WP_CORE_DIR }}/src/wp-content/plugins/amp

- name: Run Single Site tests
if: ${{ matrix.multisite == false }}
if: ${{ matrix.multisite == false && needs.pre-run.outputs.changed-php-count > 0 }}
run: |
if [[ ${{ matrix.external-http == false }} == true && ${{ matrix.coverage == false }} == true ]]; then
phpunit --verbose
Expand All @@ -436,7 +438,7 @@ jobs:
working-directory: ${{ env.WP_CORE_DIR }}/src/wp-content/plugins/amp

- name: Run multisite tests
if: ${{ matrix.multisite == true }}
if: ${{ matrix.multisite == true && needs.pre-run.outputs.changed-php-count > 0 }}
run: |
if [[ ${{ matrix.external-http == false }} == true && ${{ matrix.coverage == false }} == true ]]; then
phpunit --verbose
Expand Down
Loading

0 comments on commit 66b8668

Please sign in to comment.