Skip to content

Commit

Permalink
update compatibility for 2024 (#3)
Browse files Browse the repository at this point in the history
* laravel 9 to 11 plz

* limit test versions for php compatibility

* try with exclude

* please

* Apply suggestions from code review

* Update composer.json

* Update composer.json

* Update composer.json

* Update phpstan.neon

* Update composer.json

* Update Cronitor.php

* Update tests.yaml

---------

Co-authored-by: Will Power <[email protected]>
  • Loading branch information
cdr0y and willpower232 authored Sep 19, 2024
1 parent 458bacf commit fa556ea
Show file tree
Hide file tree
Showing 4 changed files with 32 additions and 12 deletions.
20 changes: 17 additions & 3 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,25 @@ jobs:
strategy:
fail-fast: false
matrix:
php: ["8.0", 8.1, 8.2]
illuminate: [9]
php: ["8.0", 8.1, 8.2, 8.3]
illuminate: [9, 10, 11]
exclude:
# Exclude incompatible PHP versions for Laravel 9
- php: 8.3
illuminate: 9

# Exclude incompatible PHP versions for Laravel 10
- php: "8.0"
illuminate: 10

# Exclude incompatible PHP versions for Laravel 11
- php: "8.0"
illuminate: 11
- php: 8.1
illuminate: 11

steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4

- name: "Run tests on PHP: ${{ matrix.php }}, Illuminate: ${{ matrix.illuminate }}"
run: docker build . --build-arg PHP_VERSION=${{ matrix.php }} --build-arg ILLUMINATE_VERSION=${{ matrix.illuminate }}
Expand Down
17 changes: 11 additions & 6 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@
}
],
"require": {
"illuminate/support": "^9.0",
"illuminate/contracts": "^9.0",
"illuminate/support": "^9.0|^10.0|^11.0",
"illuminate/contracts": "^9.0|^10.0|^11.0",
"synergitech/cronitor": "^0.0.1"
},
"autoload": {
Expand All @@ -26,12 +26,17 @@
]
}
},
"config": {
"allow-plugins": {
"phpstan/extension-installer": true
}
},
"require-dev": {
"symplify/easy-coding-standard": "^9.2",
"phpunit/phpunit": "^9.5",
"phpstan/phpstan": "^0.12.84",
"phpunit/phpunit": "^9.5|^10.0",
"phpstan/extension-installer": "^1.1",
"php-parallel-lint/php-parallel-lint": "^1.3",
"nunomaduro/larastan": "^0.7.4",
"orchestra/testbench": "^7.0"
"larastan/larastan": "^2.9",
"orchestra/testbench": "^7.0|^8.0|^9.0"
}
}
3 changes: 0 additions & 3 deletions phpstan.neon
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
includes:
- ./vendor/nunomaduro/larastan/extension.neon

parameters:
paths:
- src
Expand Down
4 changes: 4 additions & 0 deletions src/Facades/Cronitor.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@
use Illuminate\Support\Facades\Facade;
use SynergiTech\Cronitor\Laravel\Cronitor as AccessorClass;

/**
* @method static bool telemetryEnabled()
* @method static mixed monitorJob(string $monitorKey, callable $job)
*/
class Cronitor extends Facade
{
protected static function getFacadeAccessor()
Expand Down

0 comments on commit fa556ea

Please sign in to comment.