Merge pull request #64 from FriendsOfPHP/up

Drop support for PHP < .8.0 + Add support for PHP 8.2, and 8.3

Author: lyrixx

.github/workflows/ci.yml

@@ -2,10 +2,8 @@ name: CI
 
 on:
   push:
-    branches: [master]
+    branches: [main]
   pull_request:
-  schedule:
-    - cron: "0 0 * * MON"
 
 jobs:
   php-cs-fixer:
@@ -38,17 +36,17 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php-version: ["7.4", "8.0", "8.1"]
+        php-version: ["8.2", "8.3"]
         composer-flags: [""]
         name: [""]
         include:
-          - php-version: 7.4
+          - php-version: 8.1
             composer-flags: "--prefer-lowest"
             name: "(prefer lowest dependencies)"
 
     services:
       service-name-1:
-        image: consul
+        image: consul:1.15
         ports:
           - 8500:8500
 

.php-cs-fixer.php

@@ -10,7 +10,7 @@
 return (new PhpCsFixer\Config())
     ->setRiskyAllowed(true)
     ->setRules([
-        '@PHP74Migration' => true,
+        '@PHP81Migration' => true,
         '@PhpCsFixer' => true,
         '@Symfony' => true,
         '@Symfony:risky' => true,

CHANGELOG.md

@@ -1,6 +1,12 @@
 # CHANGELOG
 
-## 5.2.0 (not released yet)
+## 5.3.0 (not released yet)
+
+## 5.2.0 (2024-03-04)
+
+* Drop support for PHP < .8.0
+* Add support for PHP 8.2, and 8.3
+* Drop support for Symfony < 5.4, and 6.0, 6.1, 6.2, and 6.3
 
 ## 5.1.0 (2023-12-21)
 

composer.json

@@ -11,12 +11,12 @@
         }
     ],
     "require": {
-        "php": ">=7.4",
+        "php": ">=8.1",
         "psr/log": "^1|^2|^3",
-        "symfony/http-client": "^5.1|^6.0|^7.0"
+        "symfony/http-client": "^5.4|^6.4|^7.0"
     },
     "require-dev": {
-        "symfony/phpunit-bridge": "^5.0"
+        "symfony/phpunit-bridge": "^6.0|^7.0"
     },
     "autoload": {
         "psr-4": {

src/Client.php

@@ -16,7 +16,7 @@ final class Client implements ClientInterface
     private HttpClientInterface $client;
     private LoggerInterface $logger;
 
-    public function __construct(array $options = [], LoggerInterface $logger = null, HttpClientInterface $client = null)
+    public function __construct(array $options = [], ?LoggerInterface $logger = null, ?HttpClientInterface $client = null)
     {
         if (!$client) {
             $options['base_uri'] = DsnResolver::resolve($options);
@@ -27,7 +27,7 @@ public function __construct(array $options = [], LoggerInterface $logger = null,
         $this->logger = $logger ?? new NullLogger();
     }
 
-    public function get(string $url = null, array $options = []): ConsulResponse
+    public function get(?string $url = null, array $options = []): ConsulResponse
     {
         return $this->doRequest('GET', $url, $options);
     }

src/ClientInterface.php

@@ -4,7 +4,7 @@
 
 interface ClientInterface
 {
-    public function get(string $url = null, array $options = []): ConsulResponse;
+    public function get(?string $url = null, array $options = []): ConsulResponse;
 
     public function head(string $url, array $options = []): ConsulResponse;
 

src/Helper/LockHandler.php

@@ -14,7 +14,7 @@ final class LockHandler
 
     private $sessionId;
 
-    public function __construct($key, $value = null, Session $session = null, KV $kv = null)
+    public function __construct($key, $value = null, ?Session $session = null, ?KV $kv = null)
     {
         $this->key = $key;
         $this->value = $value;

src/Helper/MultiLockHandler.php

@@ -4,7 +4,6 @@
 
 use Consul\Services\KV;
 use Consul\Services\Session;
-use Exception;
 
 class MultiLockHandler
 {
@@ -45,7 +44,7 @@ public function lock(): bool
 
                 $lockedResources[] = $resource;
             }
-        } catch (Exception $e) {
+        } catch (\Exception $e) {
             $result = false;
         } finally {
             if (!$result) {

src/Helper/MultiSemaphore.php

@@ -5,7 +5,6 @@
 use Consul\Helper\MultiSemaphore\Resource;
 use Consul\Services\KV;
 use Consul\Services\Session;
-use RuntimeException;
 
 class MultiSemaphore
 {
@@ -34,7 +33,7 @@ public function getResources(): array
     public function acquire(): bool
     {
         if (null !== $this->sessionId) {
-            throw new RuntimeException('Resources are acquired already');
+            throw new \RuntimeException('Resources are acquired already');
         }
 
         // Start a session

src/Services/Agent.php

@@ -11,7 +11,7 @@ final class Agent
 {
     private ClientInterface $client;
 
-    public function __construct(ClientInterface $client = null)
+    public function __construct(?ClientInterface $client = null)
     {
         $this->client = $client ?: new Client();
     }