Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Maintenance] Simplify GitHub action workflow #446

Merged
merged 1 commit into from
Apr 26, 2022
Merged

[Maintenance] Simplify GitHub action workflow #446

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 9 additions & 9 deletions .github/workflows/build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ on:
jobs:
tests:
runs-on: ubuntu-latest
name: "PHP ${{ matrix.php }}${{ matrix.symfony != '' && format(', Symfony {0}', matrix.symfony) || '' }}${{ matrix.twig != '' && format(', Twig {0}', matrix.twig) || '' }}${{ matrix.winzou != '' && format(', Winzou {0}', matrix.winzou) || '' }}${{ matrix.composer-flags != '' && format(', Composer with {0}', matrix.composer-flags) || '' }}"
name: "PHP ${{ matrix.php }}, Symfony ${{ matrix.symfony }}, Twig ${{ matrix.twig }}"
strategy:
fail-fast: false
matrix:
Expand Down Expand Up @@ -68,8 +68,8 @@ jobs:
-
name: Install dependencies
run: |
composer update ${{ matrix.composer-flags }} --no-scripts
(cd src/Component && composer update ${{ matrix.composer-flags }} --no-scripts)
composer update --no-scripts
(cd src/Component && composer update --no-scripts)

-
name: Prepare test application
Expand Down Expand Up @@ -114,22 +114,22 @@ jobs:
-
name: Run smoke tests without friendsofsymfony/rest-bundle willdurand/hateoas-bundle jms/serializer-bundle packages
run: |
composer remove friendsofsymfony/rest-bundle willdurand/hateoas-bundle jms/serializer-bundle --no-scripts ${{ matrix.composer-flags }}
composer remove friendsofsymfony/rest-bundle willdurand/hateoas-bundle jms/serializer-bundle --no-scripts
(cd src/Bundle/test && bin/console cache:clear --env=test_without_fosrest)
composer require friendsofsymfony/rest-bundle willdurand/hateoas-bundle jms/serializer-bundle --no-scripts ${{ matrix.composer-flags }}
composer require friendsofsymfony/rest-bundle willdurand/hateoas-bundle jms/serializer-bundle --no-scripts

-
name: Run lint container without winzou/state-machine-bundle package
run: |
composer remove winzou/state-machine-bundle --no-scripts ${{ matrix.composer-flags }}
composer remove winzou/state-machine-bundle --no-scripts
(cd src/Bundle/test && bin/console cache:clear --env=test_without_state_machine)
(cd src/Bundle/test && bin/console lint:container --env=test_without_state_machine)
composer require winzou/state-machine-bundle --no-scripts ${{ matrix.composer-flags }}
composer require winzou/state-machine-bundle --no-scripts

-
name: Run lint container without twig/twig package
run: |
composer remove symfony/twig-bundle --no-scripts ${{ matrix.composer-flags }}
composer remove symfony/twig-bundle --no-scripts
(cd src/Bundle/test && bin/console cache:clear --env=test_without_twig)
(cd src/Bundle/test && bin/console lint:container --env=test_without_twig)
composer require symfony/twig-bundle --no-scripts ${{ matrix.composer-flags }}
composer require symfony/twig-bundle --no-scripts