monicahq/monica

Create a command to execute all commands that we need to run before pushing final commit to a PR

Open

#1.158 geöffnet am 12. Apr. 2018

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)PHP (2.464 Forks)batch import
feature requesthelp wantedproductivity tools

Repository-Metriken

Stars
 (24.641 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Each time we need to push our final commit to a PR, we need to

  • php artisan lang:generate
  • npm run prod

And sometimes other stuff too.

Perhaps we could create a command php artisan finalize:pr or something that would do that for us.

Contributor Guide

Research-Richtung
Das Issue fordert einen neuen Artisan Befehl, der `php artisan lang:generate` und `npm run prod` ausführt. Zur Implementierung sollten Sie vorhandene Befehle im Verzeichnis `app/Console/Commands` betrachten, um das Muster zu verstehen. Um andere Artisan Befehle auszuführen, verwenden Sie die `Artisan::call()` Fassade. Der Befehl sollte in `app/Console/Kernel.php` registriert werden. Beachten Sie außerdem, dass der Befehl zusätzliche Argumente akzeptieren könnte, um weitere Schritte auszuführen. Der Zugewiesene ist derzeit zugeteilt, koordinieren Sie daher mit ihm, bevor Sie beginnen.
Tech Stack
phpjavascript
Domain
clideveloper experience
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Blockiert
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Basic Laravel structureFamiliarity with Artisan commands
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60