Avoid the extra PHP process in the wp launcher
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 65/100
Rechercherichtung
Beginne damit, bin/wp in wp-cli/wp-cli zu lesen, um den php -r "echo PHP_OS;" Aufruf zu finden, der für die Cygwin-Erkennung verwendet wird. Entwirf eine shell-basierte Prüfung, um den zusätzlichen PHP-Prozess zu vermeiden. Erledigt bedeutet, dass wp mit einem PHP-Aufruf startet und die hier aufgezeichneten CI-Zeitmessungen eine Beschleunigung gegenüber der #384-Baseline zeigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
The wp launcher script starts PHP twice for every command: once to run php -r "echo PHP_OS;", which is only needed to detect Cygwin, and once to run WP-CLI itself. Together with the shell wrappers this adds about 30 ms to every call (121 ms versus 92 ms for wp cli version locally).
The test suites make well over 10,000 wp calls per run of the bundled commands, so this adds up to several minutes for each CI configuration. It also slows down every real-world invocation.
The fix belongs in bin/wp in wp-cli/wp-cli. It is tracked here because the test suite is where the impact is largest.
Measuring: take the baseline from #384, then open a draft PR, run the same CI jobs against it and record both results here before we decide.
- Vorherrschende Sprache
- PHP
- Sterne
- 42
- Forks
- 28
- Ø Merge
- 17 Std. 14 Min.
- Gemergte PRs (30 T.)
- 11
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus wp-cli/wp-cli-tests
-
scope:testing
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 20/100
wp-cli/wp-cli-tests#395 ·
Maintainer antworten meist innerhalb von 1 Tag
-
scope:testing
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 30/100
wp-cli/wp-cli-tests#394 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug scope:testing
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
wp-cli/wp-cli-tests#393 ·
Maintainer antworten meist innerhalb von 1 Tag
-
scope:testing
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 70/100
wp-cli/wp-cli-tests#392 ·
Maintainer antworten meist innerhalb von 1 Tag
-
scope:testing status:unconfirmed
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
wp-cli/wp-cli-tests#391 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in wp-cli/wp-cli-tests
Ähnliche Issues
-
jira-created
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 2 Tagen
-
enhancement
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 78/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
grokability/snipe-it#19996 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
Add a dark themeOffengood first issue laravel-appointment-reminders
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100