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.

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

Updated composer.json, .travis.yml, and made Travis CI builds pass #14

Merged
merged 4 commits into from Jun 4, 2019

Conversation

Projects
None yet
2 participants
@user00265
Copy link
Contributor

commented Jun 4, 2019

I have updated composer.json to get all the latest versions of dependencies (incl. require-dev), and updated PHP requirement to PHP 7.2, since braintree/braintree_php requires that as a minimum.

I have also updated .travis.yml to reflect the minimum PHP version, updated it to match the files that @spatie uses for their Travis CI builds, and the project now passes builds all the way to PHP 7.4snapshot.

Build Status

user00265 added some commits Jun 4, 2019

Renamed config/laravel-braintree.php -> config/braintree.php
 - Source code actually calls for braintree.php and not laravel-braintree.php

Signed-off-by: Elisamuel Resto <sam@simplysam.us>
Updated composer.json dependencies (incl. require-dev)
 - php updated to 7.2
 - braintree/braintree_php updated to 4.1; requires PHP 7.2 or newer
 - illuminate/support updated to 5.8 (Laravel 5.8)
 - graham-campbell/manager updated to 4.2

require-dev
 - graham-campbell/testbench updated to 5.2
 - mockery/mockery updated to 1.2
 - phpunit/phpunit updated to 8.1

Signed-off-by: Elisamuel Resto <sam@simplysam.us>
Updated .travis.yml
 - Modeleted after @spatie repositories
 - Added -vvv due to issues with timeouts
 - Added travis_wait due to issues with timeouts
 - Removed PHP 7.1 since braintree/braintree_php v4 requires PHP 7.2 minimum
 - Added cache entries
 - Only build against master
 - Added PHP 7.4snapshot build, with "allow_failure"

Signed-off-by: Elisamuel Resto <sam@simplysam.us>
Updated .travis.yml
 - Removed --prefer-lowest option from composer install (not an option)

@faustbrian faustbrian merged commit 831ca9f into artisanry:master Jun 4, 2019

user00265 added a commit to user00265/braintree that referenced this pull request Jun 6, 2019

Fix src/BraintreeManager.php due to 831ca9f
 - Renamed 'laravel-braintree' to 'braintree'
 - Introduced by artisanry#14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.