Skip to content

chore: phpcs using config in github actions, restore: phpcs.xml.dist #26

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Nov 20, 2025
+63 −1
Merged

chore: phpcs using config in github actions, restore: phpcs.xml.dist #26

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
2b8a3c0
chore: add php-cs-fixer for github workflows (pipelines)
Nov 19, 2025
c243d50
restore: phpcs.xml.dist
RobinTheHood Nov 14, 2025
27ec84d
Revert "chore: add php-cs-fixer for github workflows (pipelines)"
Nov 19, 2025
2b1dfeb
chore: explicit usage configuration file phpcs.xml.dist
Nov 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/php-code-style.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,4 +30,4 @@ jobs:
# admin/*.php = Prüft alle PHP-Dateien im admin-Ordner
# includes/*.php = Prüft alle PHP-Dateien im includes-Ordner
- name: Check PHP Code Style (PSR-12)
run: phpcs --standard=PSR12 *.php admin/*.php includes/*.php
run: phpcs --standard=phpcs.xml.dist *.php admin/*.php includes/*.php
62 changes: 62 additions & 0 deletions phpcs.xml.dist
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
<?xml version="1.0"?>
<ruleset name="Project PSR-12 Style Only">
<description>
PSR-12 als Basis, aber alle „logiknahen“ Regeln (Namespaces, Side Effects,
Strict Types, Typ-Decls etc.) sind deaktiviert. Es bleiben vor allem
Whitespace- und Formatierungsregeln übrig.
</description>

<!-- Was nie geprüft werden soll -->
<exclude-pattern>includes/external/*</exclude-pattern>

<!-- Basis: kompletter PSR-12 Standard -->
<rule ref="PSR12"/>

<!-- =========
LOGIK-/STRUKTUR-REGELN DEAKTIVIEREN
========= -->

<!-- Side Effects & "1 Klasse pro Datei + Namespace" -->
<rule ref="PSR1.Files.SideEffects">
<severity>0</severity>
</rule>

<rule ref="PSR1.Classes.ClassDeclaration">
<severity>0</severity>
</rule>

<!-- declare(strict_types=1) & Co. -->
<rule ref="PSR12.Files.DeclareStatement">
<severity>0</severity>
</rule>

<!-- Typed Parameter & Return Types -->
<rule ref="PSR12.Functions.NullableTypeDeclaration">
<severity>0</severity>
</rule>

<rule ref="PSR12.Functions.ReturnTypeDeclaration">
<severity>0</severity>
</rule>

<!-- Kurzformen von Typkeywords (bool/int/...) -->
<rule ref="PSR12.Keywords.ShortFormTypeKeywords">
<severity>0</severity>
</rule>

<!-- Namespace-Struktur -->
<rule ref="PSR12.Namespaces.CompoundNamespaceDepth">
<severity>0</severity>
</rule>

<!-- Sichtbarkeit von Konstanten erzwingen (kann Backward Compatibility sein) -->
<rule ref="PSR12.Properties.ConstantVisibility">
<severity>0</severity>
</rule>

<!-- Optional: Wenn du noch mehr "Logik-Nähe" rauswerfen willst,
kannst du hier weitere Sniffs nach Bedarf auf severity=0 setzen. -->

<!-- Nur Infos: Warnungen kannst du komplett ausblenden, wenn du magst -->
<!-- <arg name="warning-severity" value="0"/> -->
</ruleset>