Skip to content

Commit

Permalink
minor #1493 Update tests.yaml (seb-jean)
Browse files Browse the repository at this point in the history
This PR was merged into the main branch.

Discussion
----------

Update tests.yaml

Hi,

I updated the workflow to have the same as the one proposed in the blog article: [Upgrading Symfony Websites to AssetMapper
](https://symfony.com/blog/upgrading-symfony-websites-to-assetmapper#changes-in-ci-and-deployment)
What do you think ?

Commits
-------

1ab3e52 Update tests.yaml
  • Loading branch information
javiereguiluz committed Feb 16, 2024
2 parents b3aee5f + 1ab3e52 commit bbc5de3
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,11 @@ jobs:
- name: "Install dependencies"
run: composer install --ansi --no-interaction --no-progress

- name: "Build Sass"
run: php bin/console sass:build
- name: "Build and compile assets"
run: |
php bin/console importmap:install
php bin/console sass:build
php bin/console asset-map:compile
- name: "Install PHPUnit"
run: vendor/bin/simple-phpunit install
Expand Down

0 comments on commit bbc5de3

Please sign in to comment.