Bugfix - Self referential foreign key creating separate migration (#87)

* Fix self referential constraint being created as a separate file

* Pull in laravel pint, use it for formatting

* Add test

* Add a matrix for testing

* Include php version in keys

Author: bennett-treptow

.github/workflows/php.yml

@@ -8,6 +8,9 @@ on:
 
 jobs:
   build:
+    strategy:
+      matrix:
+        php: ['7.4', '8.0', '8.1']
     runs-on: ubuntu-latest
 
     steps:
@@ -16,22 +19,25 @@ jobs:
     - name: Setup PHP
       uses: shivammathur/setup-php@v2
       with:
-        php-version: '7.4'
+        php-version: ${{ matrix.php }}
 
     - name: Cache Composer packages
       id: composer-cache
       uses: actions/cache@v2
       with:
         path: vendor
-        key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
+        key: ${{ runner.os }}-php-${{ matrix.php }}-${{ hashFiles('**/composer.lock') }}
         restore-keys: |
-          ${{ runner.os }}-php-
+          ${{ runner.os }}-php-${{ matrix.php }}-
+
+    - name: Remove pint if not PHP 8.1
+      run: |
+        if [ '${{ matrix.php }}' != '8.1' ]; then
+          composer remove laravel/pint --dev
+        fi
 
     - name: Install dependencies
       run: composer install --prefer-dist --no-progress
 
-    # Add a test script to composer.json, for instance: "test": "vendor/bin/phpunit"
-    # Docs: https://getcomposer.org/doc/articles/scripts.md
-
     - name: Run test suite
       run: composer run test

.php_cs

@@ -29,7 +29,7 @@
     },
     "require-dev": {
         "orchestra/testbench": "^6.17|^8.0|^9.0",
-        "friendsofphp/php-cs-fixer": "^2.16|^3.14"
+        "laravel/pint": "^1.15"
     },
     "scripts": {
         "post-autoload-dump": [
@@ -38,7 +38,7 @@
         "test": [
             "vendor/bin/phpunit"
         ],
-        "lint": "vendor/bin/php-cs-fixer fix"
+        "lint": "vendor/bin/pint"
     },
     "extra": {
         "laravel": {
@@ -48,4 +48,4 @@
         }
     },
     "prefer-stable": true
-}
+}