Konfiguriert phpcs sniffs etwas effektiver

[datengraben]

Konfiguriert phpcs folgendermaßen:

• plugin prefix für globals und i18n calls
• kosmetische sniffs wie fullstop bei inline comments oder snakeCase für member Variablen und Funktionen
• Hebt einen guten Teil der Compat Sniffs, alle Wordpress-Sniffs und die für normale Kommentare für die Tests auf
• hebt die compat php testVersion auf 7.3 (oder unterstützt man wirklich noch niedrigere Versionen?) Andere Plugins machen es auch so also alles zwischen 7.1 und 7.4 dabei.
• hebt die compat wp testversion auf 5.0 (habe bei den Plugins geschaut Yoast, woocommerce und wp-graphql machen es auch alle so). Generell kann mich sich hier auch die Frage stellen ob man WP-Features von einer Version benötigt oder mit einer Version Wordpress-Features verhindert.
• Konfiguriert den lokalen Cache

Für mich zielt das auf #1160 und #1209 um da einheitlich zu sein und sich an den best practices zu orientieren.

Aus meiner Sicht so erstmal fertig aber Feedback oder Ergänzung gerne.

[hansmorb]

Bin mir unsicher, ob es nur meine kaputte Konfiguration ist aber ich kriege so immer noch den Fehler.

Oder hier wenn ich versuche Einstellungen zu ändern:

[datengraben]

Das liegt an #1200. Vor dem autoload der composer Definition sind die wordpress Funktionen nicht verfügbar.
Da gibt es verschiedene Wege, abhängig von der IDE. In PHP-Storm kannst du ein Wordpress-Core-Verzeichnis konfigurieren, das sollte den Fehler eigentlich beheben. (Wenn das nicht geht kann ich dir meine schicken).

Invocation via CLI als hack: Entweder tests/bootstrap.php als Pfad in die composer - autoloader Liste aufnehmen oder die includes-Pfade aus dem autoloader komplett rausnehmen. Letzteres mache ich, auch da ich den tests/bootstrap.php Eintrag etwas unschön finde. Bei mir funktionieren dann zumindest die Aufrufe der Tools aus dem vendor Verzeichnis, gleichzeitig fehlen beim Aufruf von phpunit aber die Funktionen der PHP-Files aus dem includes-Ordner in unserem Test-Code (da wechsel ich dann immer manuell)

Ich sehe keine andere Lösung, so lange wir die PHP-Definitionen des includes Ordners in seiner jetzigen Form beibehalten. Bzw. hatte mir bisher noch keine Zeit genommen das mit dem Setup z.B. von woocommerce und yoastseo zu vergleichen, auch da es mit den Hacks ja bei mir geht.

[hansmorb]

Ja, das ist meine Einstellung:

[datengraben]

@hansmorb Das ist meine Konfiguration in PHP-Storm

Das ist die einzige Wordpress-Integration für PHP-Storm die ich nutze (den installation path konfiguriere ich nicht)

Das habe ich auf diesem Branch noch zusätzlich installiert. composer require --dev dealerdirect/phpcodesniffer-composer-installer

Dann ging es bei mir.

[hansmorb]

@datengraben Ich kriege leider immer noch den gleichen Fehler. Bei dem composer require --dev phpcompatibility/phpcompatibility-wp habe ich noch zusätzlich nach composer install diesen Fehler bekommen:

Failed to set PHP CodeSniffer installed_paths Config

Hat es vielleicht was damit zu tun?

[datengraben]

@datengraben Ich kriege leider immer noch den gleichen Fehler. Bei dem composer require --dev phpcompatibility/phpcompatibility-wp habe ich noch zusätzlich nach composer install diesen Fehler bekommen:

Failed to set PHP CodeSniffer installed_paths Config

Hat es vielleicht was damit zu tun?

@hansmorb es sollte composer require --dev dealerdirect/phpcodesniffer-composer-installer im vorherigen Comment heißen.

[hansmorb]

Hmm, bei mir klappt das irgendwie immer noch nicht. Aber wir können das meinetwegen trotzdem gerne mergen, wenn es dir bei der Entwicklung hilft ist das ja schonmal ein Gewinn :)