Enhancement: Executed rector PHP 8.2 set (#244)

* Enhancement: Executed rector PHP 8.2 set

* Fix: Psalm

Author: OskarStark

.github/workflows/ci.yml

@@ -75,7 +75,7 @@ jobs:
         with:
           php-version: 8.3
           coverage: none
-          tools: php-cs-fixer:3.42, cs2pr
+          tools: php-cs-fixer:3.64, cs2pr
 
       - name: Checkout code
         uses: actions/checkout@v4

config/packages/doctrine.yaml

@@ -12,7 +12,7 @@ doctrine:
         mappings:
             App:
                 is_bundle: false
-                type: annotation
+                type: attribute
                 dir: '%kernel.project_dir%/src/Entity'
                 prefix: 'App\Entity'
                 alias: App

public/config.php

@@ -4,15 +4,12 @@
     exit("This script cannot be run from the CLI. Run it from a browser.\n");
 }
 
-if (!in_array(@$_SERVER['REMOTE_ADDR'], array(
-    '127.0.0.1',
-    '::1',
-))) {
+if (!in_array(@$_SERVER['REMOTE_ADDR'], ['127.0.0.1', '::1'])) {
     header('HTTP/1.0 403 Forbidden');
     exit('This script is only accessible from localhost.');
 }
 
-require_once dirname(__FILE__).'/../app/SymfonyRequirements.php';
+require_once __DIR__.'/../app/SymfonyRequirements.php';
 
 $symfonyRequirements = new SymfonyRequirements();
 

public/index.php

@@ -4,6 +4,4 @@
 
 require_once dirname(__DIR__).'/vendor/autoload_runtime.php';
 
-return function (array $context) {
-    return new Kernel($context['APP_ENV'], (bool) $context['APP_DEBUG']);
-};
+return fn(array $context) => new Kernel($context['APP_ENV'], (bool) $context['APP_DEBUG']);

src/Api/Issue/GithubIssueApi.php

@@ -10,19 +10,13 @@
 
 class GithubIssueApi implements IssueApi
 {
-    private $resultPager;
-    private $issueCommentApi;
-    private $issueApi;
-    private $searchApi;
-    private $botUsername;
-
-    public function __construct(ResultPager $resultPager, Comments $issueCommentApi, Issue $issueApi, Search $searchApi, string $botUsername)
-    {
-        $this->resultPager = $resultPager;
-        $this->issueCommentApi = $issueCommentApi;
-        $this->issueApi = $issueApi;
-        $this->searchApi = $searchApi;
-        $this->botUsername = $botUsername;
+    public function __construct(
+        private readonly ResultPager $resultPager,
+        private readonly Comments $issueCommentApi,
+        private readonly Issue $issueApi,
+        private readonly Search $searchApi,
+        private readonly string $botUsername,
+    ) {
     }
 
     public function open(Repository $repository, string $title, string $body, array $labels)

src/Api/Label/GithubLabelApi.php

@@ -20,34 +20,14 @@ class GithubLabelApi implements LabelApi
      *
      * @var array<array-key, array<array-key, bool>>
      */
-    private $labelCache = [];
-
-    /**
-     * @var Labels
-     */
-    private $labelsApi;
-
-    /**
-     * @var ResultPager
-     */
-    private $resultPager;
-
-    /**
-     * @var CacheInterface
-     */
-    private $cache;
-
-    /**
-     * @var LoggerInterface
-     */
-    private $logger;
-
-    public function __construct(Labels $labelsApi, ResultPager $resultPager, CacheInterface $cache, LoggerInterface $logger)
-    {
-        $this->labelsApi = $labelsApi;
-        $this->resultPager = $resultPager;
-        $this->cache = $cache;
-        $this->logger = $logger;
+    private array $labelCache = [];
+
+    public function __construct(
+        private readonly Labels $labelsApi,
+        private readonly ResultPager $resultPager,
+        private readonly CacheInterface $cache,
+        private readonly LoggerInterface $logger,
+    ) {
     }
 
     public function getIssueLabels($issueNumber, Repository $repository): array

src/Api/Milestone/GithubMilestoneApi.php

@@ -11,25 +11,15 @@
  */
 class GithubMilestoneApi implements MilestoneApi
 {
-    /**
-     * @var Milestones
-     */
-    private $milestonesApi;
-
-    /**
-     * @var Issue
-     */
-    private $issuesApi;
-
     /**
      * @var string[][]
      */
-    private $cache = [];
+    private array $cache = [];
 
-    public function __construct(Milestones $milestonesApi, Issue $issuesApi)
-    {
-        $this->milestonesApi = $milestonesApi;
-        $this->issuesApi = $issuesApi;
+    public function __construct(
+        private readonly Milestones $milestonesApi,
+        private readonly Issue $issuesApi,
+    ) {
     }
 
     private function getMilestones(Repository $repository): array
@@ -48,7 +38,7 @@ private function getMilestones(Repository $repository): array
         return $this->cache[$key];
     }
 
-    public function updateMilestone(Repository $repository, int $issueNumber, string $milestoneName)
+    public function updateMilestone(Repository $repository, int $issueNumber, string $milestoneName): void
     {
         $milestoneNumber = null;
         foreach ($this->getMilestones($repository) as $milestone) {
@@ -77,7 +67,7 @@ public function exists(Repository $repository, string $milestoneName): bool
         return false;
     }
 
-    private function getCacheKey(Repository $repository)
+    private function getCacheKey(Repository $repository): string
     {
         return sprintf('%s_%s', $repository->getVendor(), $repository->getName());
     }

src/Api/Milestone/MilestoneApi.php

@@ -9,7 +9,7 @@
  */
 interface MilestoneApi
 {
-    public function updateMilestone(Repository $repository, int $issueNumber, string $milestoneName);
+    public function updateMilestone(Repository $repository, int $issueNumber, string $milestoneName): void;
 
     public function exists(Repository $repository, string $milestoneName): bool;
 }

src/Api/Milestone/NullMilestoneApi.php

@@ -8,7 +8,7 @@
 
 class NullMilestoneApi implements MilestoneApi
 {
-    public function updateMilestone(Repository $repository, int $issueNumber, string $milestoneName)
+    public function updateMilestone(Repository $repository, int $issueNumber, string $milestoneName): void
     {
     }
 

src/Api/PullRequest/GithubPullRequestApi.php

@@ -11,21 +11,18 @@
  */
 class GithubPullRequestApi implements PullRequestApi
 {
-    private $pullRequest;
-    private $search;
-
-    public function __construct(PullRequest $pullRequest, Search $search)
-    {
-        $this->pullRequest = $pullRequest;
-        $this->search = $search;
+    public function __construct(
+        private readonly PullRequest $pullRequest,
+        private readonly Search $search,
+    ) {
     }
 
     public function show(Repository $repository, $number): array
     {
         return (array) $this->pullRequest->show($repository->getVendor(), $repository->getName(), $number);
     }
 
-    public function updateTitle(Repository $repository, $number, string $title, string $body = null): void
+    public function updateTitle(Repository $repository, $number, string $title, ?string $body = null): void
     {
         $params = ['title' => $title];