Skip to content

Commit

Permalink
Remove psalm from most installations
Browse files Browse the repository at this point in the history
As Psalm is only used in one single job it can be removed in general
and then reinstalled when it is actually required before it is
executed.

This process allows to keep Psalm in the dev-dependencies so that
constributors still have it available but in CI-runs it needs not
to be installed when it is not executed.
  • Loading branch information
heiglandreas committed Oct 19, 2024
1 parent f16121a commit 99a9ccd
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
4 changes: 4 additions & 0 deletions .github/actions/setup-build/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,10 @@ runs:
shell: bash
run: composer require ${{ inputs.composer-require }} --dev --no-update --working-dir=${{ inputs.composer-working-dir }}

- name: "Remove Psalm from dependencies"
shell: bash
run: composer remove --dev vimeo/psalm

- name: "Validate composer.json"
shell: bash
run: composer validate --no-check-publish --no-check-lock --working-dir=${{ inputs.composer-working-dir }}
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,9 @@ jobs:

- name: "Psalm"
if: always()
run: vendor/bin/psalm
run: |
composer require -W --dev vimeo/psalm:^5.0
vendor/bin/psalm
unit-tests:
name: "Unit Tests - PHP ${{ matrix.php-version }}, Sf ${{ matrix.symfony-version }}${{ matrix.dependency-versions && format(', Deps: {0}', matrix.dependency-versions) }}"
Expand Down

0 comments on commit 99a9ccd

Please sign in to comment.