Merge pull request #22 from aarondfrancis/update2

Modernize PHP codebase with strict typing

Author: aarondfrancis

.github/workflows/style.yml

@@ -1,6 +1,10 @@
 name: Fix Code Styling
 
-on: [ push ]
+on:
+  push:
+    branches:
+      - main
+      - '*.x'
 
 jobs:
   lint:
@@ -18,7 +22,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: "8.4"
+          php-version: 8.4
           extensions: json, dom, curl, libxml, mbstring
           coverage: none
 

.github/workflows/tests.yml

@@ -17,16 +17,10 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ 8.1, 8.2, 8.3, 8.4 ]
+        php: [ 8.2, 8.3, 8.4 ]
         laravel: [ '10.*', '11.*', '12.*' ]
         dependency-version: [ prefer-lowest, prefer-stable ]
 
-        exclude:
-          - laravel: 11.*
-            php: 8.1
-          - laravel: 12.*
-            php: 8.1
-
     name: P${{ matrix.php }} / L${{ matrix.laravel }} / ${{ matrix.dependency-version }}
 
     steps:

CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 1.1.0 - 2025-11-27
+
+### Added
+- PHP 8.4 support
+
+### Changed
+- Default config now uses Vite instead of Laravel Mix (`vite.config.js`, `build/manifest.json`, `build/assets`)
+- Modernized codebase with strict PHP typing (return types, parameter types, property types)
+
+### Removed
+- PHP 8.1 support (EOL November 2024)
+
 ## 0.2.3 - 2022-02-15
 - Added: Support for Laravel 9
 

composer.json

@@ -10,7 +10,7 @@
         }
     ],
     "require": {
-        "php": "^8.1",
+        "php": "^8.2",
         "illuminate/support": "^10.0|^11.0|^12.0",
         "illuminate/filesystem": "^10.0|^11.0|^12.0",
         "illuminate/console": "^10.0|^11.0|^12.0"

config/airdrop.php

@@ -73,9 +73,9 @@
                 // this makes the most sense and doesn't need to be changed.
                 resource_path(),
 
-                // Any time the webpack.mix.js file is changed, it could affect the
+                // Any time the vite.config.js file is changed, it could affect
                 // the build steps, and therefore the built files.
-                base_path('webpack.mix.js'),
+                base_path('vite.config.js'),
 
                 // Depending on your package manager, you'll want to uncomment one
                 // of the following lines. Whenever JS packages are updated or
@@ -120,14 +120,11 @@
          * Files or folders that should be included.
          */
         'include' => [
-            // The mix-manifest file tells Laravel how to get your versioned assets.
-            public_path('mix-manifest.json'),
+            // The Vite manifest file tells Laravel how to get your versioned assets.
+            public_path('build/manifest.json'),
 
-            // Compiled CSS.
-            public_path('css'),
-
-            // Compiled JS.
-            public_path('js'),
+            // Compiled assets from Vite.
+            public_path('build/assets'),
         ],
 
         /*

src/AirdropServiceProvider.php

@@ -15,7 +15,7 @@
 
 class AirdropServiceProvider extends ServiceProvider
 {
-    public function boot()
+    public function boot(): void
     {
         if ($this->app->runningInConsole()) {
             $this->commands([
@@ -32,7 +32,7 @@ public function boot()
         ], 'config');
     }
 
-    public function register()
+    public function register(): void
     {
         $this->mergeConfigFrom(__DIR__ . '/../config/airdrop.php', 'airdrop');
     }

src/Commands/Debug.php

@@ -7,16 +7,16 @@
 namespace AaronFrancis\Airdrop\Commands;
 
 use AaronFrancis\Airdrop\HashGenerator;
-use Arr;
 use Illuminate\Console\Command;
+use Illuminate\Support\Arr;
 
 class Debug extends Command
 {
     protected $signature = 'airdrop:debug {--trigger=}';
 
     protected $description = 'Output the array of all triggers, or a specific trigger.';
 
-    public function handle()
+    public function handle(): void
     {
         $output = HashGenerator::make()->asArray();
 

src/Commands/Download.php

@@ -17,7 +17,7 @@ class Download extends Command
 
     protected $description = 'Run as a part of your deploy pipeline *before* assets are built.';
 
-    public function handle()
+    public function handle(): void
     {
         config([
             'airdrop.verbose' => $this->option('verbose')

src/Commands/Hash.php

@@ -15,7 +15,7 @@ class Hash extends Command
 
     protected $description = 'Output the calculated hash.';
 
-    public function handle()
+    public function handle(): void
     {
         $hash = HashGenerator::make()->generate();
 

src/Commands/Install.php

@@ -12,18 +12,8 @@
 
 class Install extends Command
 {
-    /**
-     * The name and signature of the console command.
-     *
-     * @var string
-     */
     protected $signature = 'airdrop:install';
 
-    /**
-     * The console command description.
-     *
-     * @var string
-     */
     protected $description = 'Install the Airdrop config file into your app.';
 
     public function __construct()
@@ -35,10 +25,7 @@ public function __construct()
         }
     }
 
-    /**
-     * @throws Exception
-     */
-    public function handle()
+    public function handle(): void
     {
         Artisan::call('vendor:publish', [
             '--provider' => AirdropServiceProvider::class