Skip to content

Commit 2f7ab6f

Browse files
authored
Merge pull request #143 from rollbar/fixed/ci-composer-require-stable
Fixed required minimum stability causing the CI tests to fail
2 parents 27b93b3 + 0ad823e commit 2f7ab6f

File tree

1 file changed

+8
-15
lines changed

1 file changed

+8
-15
lines changed

.github/workflows/ci.yml

Lines changed: 8 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -68,23 +68,16 @@ jobs:
6868
run: composer create-project laravel/laravel rollbar-test-app ${{ matrix.laravel }}
6969

7070
- name: Install that code using Composer rigged to look in the parent directory
71-
id: composer_require
7271
working-directory: rollbar-test-app
73-
continue-on-error: true
72+
# We arbitrarily set the version to 1.0.0, because we just need composer
73+
# to consider it to be stable.
7474
run: |
75-
composer config repositories.local '{"type":"path", "url":".."}'
76-
composer require rollbar/rollbar-laravel
77-
78-
- name: Try to resolve requirements into an installable set of packages
79-
working-directory: rollbar-test-app
80-
# This adds time, so only run this if the previous step fails.
81-
if: ${{ steps.composer_require.outcome == 'failure' }}
82-
# We are ok removing the lock file since it was generated in step 3,
83-
# before we included all our requirements.
84-
run: |
85-
rm composer.lock
86-
composer require rollbar/rollbar-laravel
87-
composer install
75+
composer config repositories.local '{
76+
"type":"path",
77+
"url":"..",
78+
"options": {"versions": {"rollbar/rollbar-laravel": "1.0.0"}}
79+
}'
80+
composer require rollbar/rollbar-laravel -W
8881
8982
- name: Setup .env
9083
working-directory: rollbar-test-app

0 commit comments

Comments
 (0)