Skip to content

Commit

Permalink
Remove redundant b24phpsdk source installation steps.
Browse files Browse the repository at this point in the history
The steps for installing b24phpsdk from source were unnecessary and have been removed from the unit tests, functional tests, and PHPStan lint workflows. This simplification should reduce maintenance overhead and speed up the workflow execution times.

Signed-off-by: mesilov <[email protected]>
  • Loading branch information
mesilov committed Dec 4, 2024
1 parent ff5fa2d commit a70123d
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 16 deletions.
5 changes: 0 additions & 5 deletions .github/workflows/lint-phpstan.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,11 +37,6 @@ jobs:
if: ${{ matrix.dependencies == 'highest' }}
run: "composer update --no-interaction --no-progress --no-suggest"

- name: "Install b24phpsdk from source"
run: |
composer remove bitrix24/b24phpsdk --no-update
composer require bitrix24/b24phpsdk --prefer-source --no-cache --ansi --no-interaction --no-progress
- name: "PHPStan"
run: "vendor/bin/phpstan --memory-limit=2G analyse"

Expand Down
5 changes: 0 additions & 5 deletions .github/workflows/tests-functional.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,6 @@ jobs:
run: |
composer update ${{ env.COMPOSER_FLAGS }}
- name: "Install b24phpsdk from source"
run: |
composer remove bitrix24/b24phpsdk --no-update
composer require bitrix24/b24phpsdk --prefer-source --no-cache --ansi --no-interaction --no-progress
- name: "Wait for PostgreSQL to be ready"
run: |
until pg_isready -h localhost -p 5432 -U user; do
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/tests-unit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,12 +35,6 @@ jobs:
- name: "Install dependencies"
run: |
composer update ${{ env.COMPOSER_FLAGS }}
composer update bitrix24/b24phpsdk --prefer-source --no-cache
- name: "Install b24phpsdk from source"
run: |
composer remove bitrix24/b24phpsdk --no-update
composer require bitrix24/b24phpsdk --prefer-source --no-cache --ansi --no-interaction --no-progress
- name: "run unit tests"
run: "php vendor/bin/phpunit --testsuite=unit_tests --display-warnings --testdox"
Expand Down

0 comments on commit a70123d

Please sign in to comment.