Replace Travis CI with GitHub Actions

ruddell · claude · ruddell · commit 9635df0ccb0b · 2026-04-16T10:22:00.000-04:00
- Add .github/workflows/tests.yml running PHP 8.2, 8.3, 8.4
- Remove .travis.yml
- Drop codecov and psalm --shepherd
- Widen illuminate/* and lumen-framework constraints to support Laravel 13
- Bump php requirement to ^7.4 to match graham-campbell/manager ^5.0
- Fix test compatibility with updated graham-campbell packages and PHPUnit 10
- Fix Psalm errors for updated package versions

Co-Authored-By: Claude Sonnet 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -0,0 +1,33 @@
+name: Tests
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        php: ['8.2', '8.3', '8.4']
+
+    name: PHP ${{ matrix.php }}
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+
+      - name: Install dependencies
+        run: composer install --no-interaction
+
+      - name: Run tests
+        run: vendor/bin/phpunit
+
+      - name: Run Psalm
+        run: vendor/bin/psalm
diff --git a/.travis.yml b/.travis.yml
diff --git a/composer.json b/composer.json
@@ -19,7 +19,7 @@
         }
     ],
     "require": {
-        "php": "^7.2|^8.0",
+        "php": "^7.4|^8.0",
         "illuminate/contracts": "^5.8|^6.0|^7.0|^8.0|^9.0|^10.0|^11.0|^12.0|^13.0",
         "illuminate/support": "^5.8|^6.0|^7.0|^8.0|^9.0|^10.0|^11.0|^12.0|^13.0",
         "graham-campbell/manager": "^5.0",
@@ -32,7 +32,7 @@
         "mockery/mockery": "^1.0",
         "phpunit/phpunit": "^7.0|^9.5.10|^10.5|^11.0",
         "psalm/plugin-laravel": "^1.4.0|^1.1|^2.7|^3.0",
-        "laravel/lumen-framework": "^5.7",
+        "laravel/lumen-framework": "^5.7|^9.0|^10.0|^11.0",
         "vimeo/psalm": "^3.17|^4.19|^5.6|^6.8"
     },
     "autoload": {
diff --git a/src/Facades/Vimeo.php b/src/Facades/Vimeo.php
@@ -25,6 +25,8 @@
  * This is the Vimeo facade class.
  *
  * @author Vincent Klaiber <hello@vinkla.com>
+ *
+ * @psalm-api
  */
 class Vimeo extends Facade
 {
@@ -33,6 +35,7 @@ class Vimeo extends Facade
      *
      * @return string
      */
+    #[\Override]
     protected static function getFacadeAccessor(): string
     {
         return 'vimeo';
diff --git a/src/VimeoFactory.php b/src/VimeoFactory.php
@@ -27,6 +27,8 @@
  * The is the Vimeo factory class.
  *
  * @author Vincent Klaiber <hello@vinkla.com>
+ *
+ * @psalm-api
  */
 class VimeoFactory
 {
diff --git a/src/VimeoManager.php b/src/VimeoManager.php
@@ -27,6 +27,8 @@
  * This is the Vimeo manager class.
  *
  * @author Vincent Klaiber <hello@vinkla.com>
+ *
+ * @psalm-api
  */
 class VimeoManager extends AbstractManager
 {
@@ -57,8 +59,11 @@ public function __construct(Repository $config, VimeoFactory $factory)
      *
      * @param array $config
      *
+     * @psalm-suppress MethodSignatureMismatch
+     *
      * @return \Vimeo\Vimeo
      */
+    #[\Override]
     protected function createConnection(array $config): Vimeo
     {
         /** @var string[] $config */
@@ -70,16 +75,13 @@ protected function createConnection(array $config): Vimeo
      *
      * @return string
      */
+    #[\Override]
     protected function getConfigName(): string
     {
         return 'vimeo';
     }
 
-    /**
-     * Get the factory instance.
-     *
-     * @return \Vimeo\Laravel\VimeoFactory
-     */
+    /** @psalm-api */
     public function getFactory(): VimeoFactory
     {
         return $this->factory;
diff --git a/src/VimeoServiceProvider.php b/src/VimeoServiceProvider.php
@@ -29,6 +29,8 @@
  * This is the Vimeo service provider class.
  *
  * @author Vincent Klaiber <hello@vinkla.com>
+ *
+ * @psalm-api
  */
 class VimeoServiceProvider extends ServiceProvider
 {
@@ -51,7 +53,7 @@ protected function setupConfig()
     {
         $source = realpath(__DIR__.'/../config/vimeo.php');
 
-        if (!$source) {
+        if ($source === false) {
             throw new \UnexpectedValueException('Could not locate config');
         }
 
@@ -69,6 +71,7 @@ protected function setupConfig()
      *
      * @return void
      */
+    #[\Override]
     public function register()
     {
         $this->registerFactory();
@@ -99,9 +102,9 @@ protected function registerManager()
     {
         $this->app->singleton('vimeo', function (Container $app) : VimeoManager {
             /** @var \Illuminate\Contracts\Config\Repository */
-            $config = $app['config'];
+            $config = $app->make('config');
             /** @var \Vimeo\Laravel\VimeoFactory */
-            $factory = $app['vimeo.factory'];
+            $factory = $app->make('vimeo.factory');
 
             return new VimeoManager($config, $factory);
         });
@@ -118,7 +121,7 @@ protected function registerBindings()
     {
         $this->app->bind('vimeo.connection', function (Container $app) : Vimeo {
             /** @var VimeoManager */
-            $manager = $app['vimeo'];
+            $manager = $app->make('vimeo');
 
             /** @var Vimeo */
             return $manager->connection();
@@ -132,6 +135,7 @@ protected function registerBindings()
      *
      * @return string[]
      */
+    #[\Override]
     public function provides(): array
     {
         return [
diff --git a/tests/AbstractTestCase.php b/tests/AbstractTestCase.php
@@ -36,7 +36,7 @@ abstract class AbstractTestCase extends AbstractPackageTestCase
      *
      * @return string
      */
-    protected function getServiceProviderClass($app)
+    protected static function getServiceProviderClass(): string
     {
         return VimeoServiceProvider::class;
     }
diff --git a/tests/AnalysisTest.php b/tests/AnalysisTest.php
@@ -37,7 +37,7 @@ class AnalysisTest extends TestCase
      *
      * @return string[]
      */
-    protected function getPaths()
+    protected static function getPaths(): array
     {
         return [
             realpath(__DIR__.'/../config'),
@@ -51,7 +51,7 @@ protected function getPaths()
      *
      * @return string[]
      */
-    protected function getIgnored()
+    protected static function getIgnored()
     {
         return [Application::class];
     }
diff --git a/tests/Facades/VimeoTest.php b/tests/Facades/VimeoTest.php
@@ -38,7 +38,7 @@ class VimeoTest extends AbstractTestCase
      *
      * @return string
      */
-    protected function getFacadeAccessor()
+    protected static function getFacadeAccessor(): string
     {
         return 'vimeo';
     }
@@ -48,7 +48,7 @@ protected function getFacadeAccessor()
      *
      * @return string
      */
-    protected function getFacadeClass()
+    protected static function getFacadeClass(): string
     {
         return Vimeo::class;
     }
@@ -58,7 +58,7 @@ protected function getFacadeClass()
      *
      * @return string
      */
-    protected function getFacadeRoot()
+    protected static function getFacadeRoot(): string
     {
         return VimeoManager::class;
     }
diff --git a/tests/VimeoFactoryTest.php b/tests/VimeoFactoryTest.php
@@ -42,23 +42,21 @@ public function testMakeStandard()
         $this->assertInstanceOf(Vimeo::class, $return);
     }
 
-    /**
-     * @expectedException \InvalidArgumentException
-     */
     public function testMakeWithoutClientId()
     {
+        $this->expectException(\InvalidArgumentException::class);
+
         $factory = $this->getVimeoFactory();
 
         $factory->make([
             'client_secret' => 'your-client-secret',
         ]);
     }
 
-    /**
-     * @expectedException \InvalidArgumentException
-     */
     public function testMakeWithoutClientSecret()
     {
+        $this->expectException(\InvalidArgumentException::class);
+
         $factory = $this->getVimeoFactory();
 
         $factory->make([

Original file line number	Diff line number	Diff line change
`@@ -25,6 +25,8 @@`
`25`	`25`	`* This is the Vimeo facade class.`
`26`	`26`	`*`
`27`	`27`	`* @author Vincent Klaiber <[email protected]>`
	`28`	`+ *`
	`29`	`+ * @psalm-api`
`28`	`30`	`*/`
`29`	`31`	`class Vimeo extends Facade`
`30`	`32`	`{`
`@@ -33,6 +35,7 @@ class Vimeo extends Facade`
`33`	`35`	`*`
`34`	`36`	`* @return string`
`35`	`37`	`*/`
	`38`	`+ #[\Override]`
`36`	`39`	`protected static function getFacadeAccessor(): string`
`37`	`40`	`{`
`38`	`41`	`return 'vimeo';`
Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,8 @@`
`27`	`27`	`* The is the Vimeo factory class.`
`28`	`28`	`*`
`29`	`29`	`* @author Vincent Klaiber <[email protected]>`
	`30`	`+ *`
	`31`	`+ * @psalm-api`
`30`	`32`	`*/`
`31`	`33`	`class VimeoFactory`
`32`	`34`	`{`
Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,8 @@`
`29`	`29`	`* This is the Vimeo service provider class.`
`30`	`30`	`*`
`31`	`31`	`* @author Vincent Klaiber <[email protected]>`
	`32`	`+ *`
	`33`	`+ * @psalm-api`
`32`	`34`	`*/`
`33`	`35`	`class VimeoServiceProvider extends ServiceProvider`
`34`	`36`	`{`
`@@ -51,7 +53,7 @@ protected function setupConfig()`
`51`	`53`	`{`
`52`	`54`	`$source = realpath(__DIR__.'/../config/vimeo.php');`
`53`	`55`
`54`		`- if (!$source) {`
	`56`	`+ if ($source === false) {`
`55`	`57`	`throw new \UnexpectedValueException('Could not locate config');`
`56`	`58`	`}`
`57`	`59`
`@@ -69,6 +71,7 @@ protected function setupConfig()`
`69`	`71`	`*`
`70`	`72`	`* @return void`
`71`	`73`	`*/`
	`74`	`+ #[\Override]`
`72`	`75`	`public function register()`
`73`	`76`	`{`
`74`	`77`	`$this->registerFactory();`
`@@ -99,9 +102,9 @@ protected function registerManager()`
`99`	`102`	`{`
`100`	`103`	`$this->app->singleton('vimeo', function (Container $app) : VimeoManager {`
`101`	`104`	`/** @var \Illuminate\Contracts\Config\Repository */`
`102`		`- $config = $app['config'];`
	`105`	`+ $config = $app->make('config');`
`103`	`106`	`/** @var \Vimeo\Laravel\VimeoFactory */`
`104`		`- $factory = $app['vimeo.factory'];`
	`107`	`+ $factory = $app->make('vimeo.factory');`
`105`	`108`
`106`	`109`	`return new VimeoManager($config, $factory);`
`107`	`110`	`});`
`@@ -118,7 +121,7 @@ protected function registerBindings()`
`118`	`121`	`{`
`119`	`122`	`$this->app->bind('vimeo.connection', function (Container $app) : Vimeo {`
`120`	`123`	`/** @var VimeoManager */`
`121`		`- $manager = $app['vimeo'];`
	`124`	`+ $manager = $app->make('vimeo');`
`122`	`125`
`123`	`126`	`/** @var Vimeo */`
`124`	`127`	`return $manager->connection();`
`@@ -132,6 +135,7 @@ protected function registerBindings()`
`132`	`135`	`*`
`133`	`136`	`* @return string[]`
`134`	`137`	`*/`
	`138`	`+ #[\Override]`
`135`	`139`	`public function provides(): array`
`136`	`140`	`{`
`137`	`141`	`return [`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ abstract class AbstractTestCase extends AbstractPackageTestCase`
`36`	`36`	`*`
`37`	`37`	`* @return string`
`38`	`38`	`*/`
`39`		`- protected function getServiceProviderClass($app)`
	`39`	`+ protected static function getServiceProviderClass(): string`
`40`	`40`	`{`
`41`	`41`	`return VimeoServiceProvider::class;`
`42`	`42`	`}`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ class AnalysisTest extends TestCase`
`37`	`37`	`*`
`38`	`38`	`* @return string[]`
`39`	`39`	`*/`
`40`		`- protected function getPaths()`
	`40`	`+ protected static function getPaths(): array`
`41`	`41`	`{`
`42`	`42`	`return [`
`43`	`43`	`realpath(__DIR__.'/../config'),`
`@@ -51,7 +51,7 @@ protected function getPaths()`
`51`	`51`	`*`
`52`	`52`	`* @return string[]`
`53`	`53`	`*/`
`54`		`- protected function getIgnored()`
	`54`	`+ protected static function getIgnored()`
`55`	`55`	`{`
`56`	`56`	`return [Application::class];`
`57`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ class VimeoTest extends AbstractTestCase`
`38`	`38`	`*`
`39`	`39`	`* @return string`
`40`	`40`	`*/`
`41`		`- protected function getFacadeAccessor()`
	`41`	`+ protected static function getFacadeAccessor(): string`
`42`	`42`	`{`
`43`	`43`	`return 'vimeo';`
`44`	`44`	`}`
`@@ -48,7 +48,7 @@ protected function getFacadeAccessor()`
`48`	`48`	`*`
`49`	`49`	`* @return string`
`50`	`50`	`*/`
`51`		`- protected function getFacadeClass()`
	`51`	`+ protected static function getFacadeClass(): string`
`52`	`52`	`{`
`53`	`53`	`return Vimeo::class;`
`54`	`54`	`}`
`@@ -58,7 +58,7 @@ protected function getFacadeClass()`
`58`	`58`	`*`
`59`	`59`	`* @return string`
`60`	`60`	`*/`
`61`		`- protected function getFacadeRoot()`
	`61`	`+ protected static function getFacadeRoot(): string`
`62`	`62`	`{`
`63`	`63`	`return VimeoManager::class;`
`64`	`64`	`}`