chore: removing support for php 7.2 (#181)

* chore: removing support for php 7.2

* ci: running php sdk tests against php 8

* chore(deps-dev): pulling squizlabs/php_codesniffer@master

* fix: restoring support for laravel 6

* fix: laravel 6 support

* ci: cleaning up the php workflow

Author: erunion

.github/workflows/php.yml

@@ -12,51 +12,8 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        php: [7.2, 7.3, 7.4]
-        laravel: [6.*, 7.*]
-        dependency-version: [prefer-lowest, prefer-stable]
-
-    name: build (${{ matrix.php }} w/ Laravel ${{ matrix.laravel }} @${{ matrix.dependency-version }})
-
-    steps:
-      - uses: actions/[email protected]
-
-      - name: Cache dependencies
-        uses: actions/cache@v2
-        with:
-          path: ~/.composer/cache/files
-          key: dependencies-laravel-${{ matrix.laravel }}-php-${{ matrix.php }}-composer-${{ hashFiles('composer.json') }}
-
-      - name: Setup PHP
-        uses: shivammathur/setup-php@v2
-        with:
-          php-version: ${{ matrix.php }}
-
-      - name: Install dependencies
-        run: |
-          composer require "laravel/framework:${{ matrix.laravel }}" --no-interaction --no-update
-          composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction --no-suggest
-
-      - name: Check code standards
-        run: composer run-script lint
-
-      - name: Execute static analysis
-        run: composer run-script psalm
-
-      - name: Run tests
-        run: composer run-script phpunit
-
-  test-laravel8:
-    runs-on: ubuntu-latest
-    defaults:
-      run:
-        working-directory: packages/php
-
-    strategy:
-      fail-fast: true
-      matrix:
-        php: [7.3, 7.4]
-        laravel: [8.*]
+        php: [7.3, 7.4, 8.0]
+        laravel: [6.*, 7.*, 8.*]
         dependency-version: [prefer-lowest, prefer-stable]
 
     name: build (${{ matrix.php }} w/ Laravel ${{ matrix.laravel }} @${{ matrix.dependency-version }})

packages/php/composer.json

@@ -14,7 +14,7 @@
     "readme"
   ],
   "require": {
-    "php": "^7.2",
+    "php": "^7.3 | ^7.4 | ^8.0",
     "illuminate/http": "^6.8 | ^7.0 | ^8.0",
     "illuminate/support": "^6.8 | ^7.0 | ^8.0",
     "ocramius/package-versions": "^1.4",
@@ -23,10 +23,10 @@
     "guzzlehttp/guzzle": "^7.0"
   },
   "require-dev": {
-    "phpunit/phpunit": "^8.5",
-    "squizlabs/php_codesniffer": "^3.5",
-    "mockery/mockery": "^1.3",
-    "vimeo/psalm": "^3.14"
+    "phpunit/phpunit": "^9.5",
+    "mockery/mockery": "^1.4",
+    "vimeo/psalm": "dev-master",
+    "squizlabs/php_codesniffer": "dev-master"
   },
   "extra": {
     "laravel": {

packages/php/tests/MetricsTest.php

@@ -133,7 +133,7 @@ public function testTrack(bool $development_mode): void
         $this->assertContains('x-documentation-url', array_keys($response->headers->all()));
         $documentation_header = array_shift($response->headers->all()['x-documentation-url']);
         $log_id = $this->getLogIdFromDocumentationHeader($documentation_header);
-        $this->assertRegExp(self::UUID_PATTERN, $log_id);
+        $this->assertMatchesRegularExpression(self::UUID_PATTERN, $log_id);
 
         // Assert that we only tracked a single request and also the payload looks as expected.
         $this->assertCount(1, $this->api_calls);
@@ -299,7 +299,7 @@ public function testConstructPayload(): void
 
         $payload_entry = $payload['request']['log']['entries'][0];
         $this->assertSame($request->url(), $payload_entry['pageref']);
-        $this->assertRegExp(
+        $this->assertMatchesRegularExpression(
             '/(\d{4}-\d{2}-\d{2})T(\d{2}:\d{2}:\d{2})\+(\d{2}:\d{2})/',
             $payload_entry['startedDateTime'],
             'startedDateTime was not in a format matching `2019-12-19T01:17:51+00:00`.'