PHP 8.4 compatibility

src/Api/Groups.php

@@ -91,7 +91,7 @@ public function show($id)
      *
      * @return mixed
      */
-    public function create(string $name, string $path, string $description = null, string $visibility = 'private', bool $lfs_enabled = null, bool $request_access_enabled = null, int $parent_id = null, int $shared_runners_minutes_limit = null)
+    public function create(string $name, string $path, ?string $description = null, string $visibility = 'private', ?bool $lfs_enabled = null, ?bool $request_access_enabled = null, ?int $parent_id = null, ?int $shared_runners_minutes_limit = null)
     {
         $params = [
             'name' => $name,
@@ -917,7 +917,7 @@ private function getSubgroupSearchResolver()
      *
      * @return mixed
      */
-    public function deployTokens($group_id, bool $active = null)
+    public function deployTokens($group_id, ?bool $active = null)
     {
         return $this->get('groups/'.self::encodePath($group_id).'/deploy_tokens', (null !== $active) ? ['active' => $active] : []);
     }

src/Api/Projects.php

@@ -466,7 +466,7 @@ public function pipelineTestReportSummary($project_id, int $pipeline_id)
      *
      * @return mixed
      */
-    public function createPipeline($project_id, string $commit_ref, array $variables = null)
+    public function createPipeline($project_id, string $commit_ref, ?array $variables = null)
     {
         $parameters = [];
 
@@ -588,7 +588,7 @@ public function allMember($project_id, int $user_id)
      *
      * @return mixed
      */
-    public function addMember($project_id, int $user_id, int $access_level, string $expires_at = null)
+    public function addMember($project_id, int $user_id, int $access_level, ?string $expires_at = null)
     {
         $params = [
             'user_id' => $user_id,
@@ -609,7 +609,7 @@ public function addMember($project_id, int $user_id, int $access_level, string $
      *
      * @return mixed
      */
-    public function saveMember($project_id, int $user_id, int $access_level, string $expires_at = null)
+    public function saveMember($project_id, int $user_id, int $access_level, ?string $expires_at = null)
     {
         $params = [
             'access_level' => $access_level,
@@ -868,7 +868,7 @@ public function enableDeployKey($project_id, int $key_id)
      *
      * @return mixed
      */
-    public function deployTokens($project_id, bool $active = null)
+    public function deployTokens($project_id, ?bool $active = null)
     {
         return $this->get($this->getProjectPath($project_id, 'deploy_tokens'), (null !== $active) ? ['active' => $active] : []);
     }

src/Api/Repositories.php

@@ -419,7 +419,7 @@ public function postCommitBuildStatus($project_id, string $sha, string $state, a
      *
      * @return mixed
      */
-    public function compare($project_id, string $fromShaOrMaster, string $toShaOrMaster, bool $straight = false, string $fromProjectId = null)
+    public function compare($project_id, string $fromShaOrMaster, string $toShaOrMaster, bool $straight = false, ?string $fromProjectId = null)
     {
         $params = [
             'from' => $fromShaOrMaster,

src/Client.php

@@ -123,7 +123,7 @@ class Client
      *
      * @return void
      */
-    public function __construct(Builder $httpClientBuilder = null)
+    public function __construct(?Builder $httpClientBuilder = null)
     {
         $this->httpClientBuilder = $builder = $httpClientBuilder ?? new Builder();
         $this->responseHistory = new History();
@@ -425,7 +425,7 @@ public function wiki(): Wiki
      *
      * @return void
      */
-    public function authenticate(string $token, string $authMethod, string $sudo = null): void
+    public function authenticate(string $token, string $authMethod, ?string $sudo = null): void
     {
         $this->getHttpClientBuilder()->removePlugin(Authentication::class);
         $this->getHttpClientBuilder()->addPlugin(new Authentication($authMethod, $token, $sudo));

src/HttpClient/Builder.php

@@ -100,10 +100,10 @@ final class Builder
      * @return void
      */
     public function __construct(
-        ClientInterface $httpClient = null,
-        RequestFactoryInterface $requestFactory = null,
-        StreamFactoryInterface $streamFactory = null,
-        UriFactoryInterface $uriFactory = null
+        ?ClientInterface $httpClient = null,
+        ?RequestFactoryInterface $requestFactory = null,
+        ?StreamFactoryInterface $streamFactory = null,
+        ?UriFactoryInterface $uriFactory = null
     ) {
         $this->httpClient = $httpClient ?? Psr18ClientDiscovery::find();
         $this->requestFactory = $requestFactory ?? Psr17FactoryDiscovery::findRequestFactory();

src/HttpClient/Plugin/Authentication.php

@@ -43,7 +43,7 @@ final class Authentication implements Plugin
      *
      * @return void
      */
-    public function __construct(string $method, string $token, string $sudo = null)
+    public function __construct(string $method, string $token, ?string $sudo = null)
     {
         $this->headers = self::buildHeaders($method, $token, $sudo);
     }
@@ -77,7 +77,7 @@ public function handleRequest(RequestInterface $request, callable $next, callabl
      *
      * @return array<string,string>
      */
-    private static function buildHeaders(string $method, string $token, string $sudo = null): array
+    private static function buildHeaders(string $method, string $token, ?string $sudo = null): array
     {
         $headers = [];
 

src/ResultPager.php

@@ -66,7 +66,7 @@ final class ResultPager implements ResultPagerInterface
      *
      * @return void
      */
-    public function __construct(Client $client, int $perPage = null)
+    public function __construct(Client $client, ?int $perPage = null)
     {
         if (null !== $perPage && ($perPage < 1 || $perPage > 100)) {
             throw new ValueError(\sprintf('%s::__construct(): Argument #2 ($perPage) must be between 1 and 100, or null', self::class));