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

Add justfile, remove coveralls, and fix AUTOLOAD in CI #1801

Merged
merged 16 commits into from
Jan 9, 2025
Merged
Prev Previous commit
Next Next commit
add more logging
  • Loading branch information
xavdid-stripe committed Jan 8, 2025
commit dc89cd151ec9c9c619b5cdedfc722e326ee1a1bb
4 changes: 3 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -126,10 +126,12 @@ jobs:
- uses: stripe/openapi/actions/stripe-mock@master

- name: Run test suite
# --yes skips the confirmation dialogue for the CI test, which modifies files
run: |
php --version
echo "autoload is: $AUTOLOAD"
just ci-test
make ci-test
just --yes ci-test

publish:
# Doesn't actually publish. The publish happens via a packagist webhook configured in the Github UI. But we still display a message here
Expand Down
1 change: 1 addition & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ test: vendor
.PHONY: test

ci-test: vendor
echo "calling build with $$AUTOLOAD"
./build.php $$AUTOLOAD
.PHONY: ci-test

Expand Down
75 changes: 31 additions & 44 deletions composer.json
Original file line number Diff line number Diff line change
@@ -1,46 +1,33 @@
{
"name": "stripe/stripe-php",
"description": "Stripe PHP Library",
"keywords": [
"stripe",
"payment processing",
"api"
],
"homepage": "https://stripe.com/",
"license": "MIT",
"authors": [
{
"name": "Stripe and contributors",
"homepage": "https://github.com/stripe/stripe-php/contributors"
"name": "stripe\/stripe-php",
"description": "Stripe PHP Library",
"keywords": [
"stripe",
"payment processing",
"api"
],
"homepage": "https:\/\/stripe.com\/",
"license": "MIT",
"authors": [
{
"name": "Stripe and contributors",
"homepage": "https:\/\/github.com\/stripe\/stripe-php\/contributors"
}
],
"require": {
"php": ">=5.6.0",
"ext-curl": "*",
"ext-json": "*",
"ext-mbstring": "*"
},
"require-dev": {
"phpunit\/phpunit": "^5.7 || ^9.0",
"friendsofphp\/php-cs-fixer": "3.5.0",
"phpstan\/phpstan": "^1.2"
},
"extra": {
"branch-alias": {
"dev-master": "2.0-dev"
}
}
],
"require": {
"php": ">=5.6.0",
"ext-curl": "*",
"ext-json": "*",
"ext-mbstring": "*"
},
"require-dev": {
"phpunit/phpunit": "^5.7 || ^9.0",
"friendsofphp/php-cs-fixer": "3.5.0",
"phpstan/phpstan": "^1.2"
},
"autoload": {
"psr-4": {
"Stripe\\": "lib/"
}
},
"autoload-dev": {
"psr-4": {
"Stripe\\": [
"tests/",
"tests/Stripe/"
]
}
xavdid-stripe marked this conversation as resolved.
Show resolved Hide resolved
},
"extra": {
"branch-alias": {
"dev-master": "2.0-dev"
}
}
}
}
1 change: 1 addition & 0 deletions justfile
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ test *options:
phpunit {{ options }}

[group('CI')]
[confirm("This will modify local and is mostly just for CI; do you want to proceed?")]
ci-test:
echo "got $AUTOLOAD"
./build.php $AUTOLOAD
Expand Down
Loading