New scripts for CI

.ci/all.sh

@@ -0,0 +1,27 @@
+#!/usr/bin/env bash
+
+#
+# all.sh
+# Copyright (c) 2024 james@firefly-iii.org
+#
+# This file is part of Firefly III (https://github.com/firefly-iii).
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as
+# published by the Free Software Foundation, either version 3 of the
+# License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with this program.  If not, see <https://www.gnu.org/licenses/>.
+#
+
+SCRIPT_DIR="$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
+
+$SCRIPT_DIR/phpcs.sh
+$SCRIPT_DIR/phpstan.sh
+$SCRIPT_DIR/phpmd.sh

.ci/php-cs-fixer/.php-cs-fixer.php

@@ -34,13 +34,35 @@ $finder = PhpCsFixer\Finder::create()
 
 $config = new PhpCsFixer\Config();
 return $config->setRules([
-                             '@PhpCsFixer:risky'           => true,
-                             '@PSR12:risky'                => true,
-                             'declare_strict_types'        => true,
-                             'strict_param'                => true,
-                             'array_syntax'                => ['syntax' => 'short'],
-                             'native_function_invocation'  => false, // annoying
-                             'php_unit_data_provider_name' => false, // bloody annoying long test names
-                             'static_lambda' => false, // breaks the Response macro for API's.
+                             'no_unused_imports'             => true,
+                             '@PhpCsFixer'                   => true,
+                             '@PHP83Migration'               => true,
+                             '@PhpCsFixer:risky'             => true,
+                             '@PSR12:risky'                  => true,
+                             'declare_strict_types'          => true,
+                             'strict_param'                  => true,
+                             'comment_to_phpdoc'             => false, // breaks phpstan lines in combination with PHPStorm.
+                             'array_syntax'                  => ['syntax' => 'short'],
+                             'native_function_invocation'    => false, // annoying
+                             'php_unit_data_provider_name'   => false, // bloody annoying long test names
+                             'static_lambda'                 => false, // breaks the Response macro for API's.
+                             'phpdoc_summary'                => false, // annoying.
+                             'single_space_around_construct' => [
+                                 'constructs_followed_by_a_single_space' => [
+                                     'protected',
+                                 ],
+                             ],
+                             'statement_indentation'         => true,
+                             'type_declaration_spaces'       => false,
+                             'cast_spaces'                   => false,
+                             'binary_operator_spaces'        => [
+                                 'default' => 'at_least_single_space',
+                                 'operators' => [
+                                     '=>' => 'align_single_space_by_scope',
+                                     '='  => 'align_single_space_minimal_by_scope',
+                                     '??='  => 'align_single_space_minimal_by_scope',
+                                 ],
+                             ],
+                             'void_return'                   => true,
                          ])
               ->setFinder($finder);