monicahq/monica

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

Open

#1 158 ouverte le 12 avr. 2018

Voir sur GitHub
 (1 commentaire) (0 réactions) (1 assigné)PHP (24 641 stars) (2 464 forks)batch import
feature requesthelp wantedproductivity tools

Description

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.

Guide contributeur

Stack technique
phpjavascript
Domaine
clideveloper experience
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
blocked
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic Laravel structureFamiliarity with Artisan commands
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
The issue requests a new Artisan command that runs `php artisan lang:generate` and `npm run prod`. To implement, look at existing commands in the `app/Console/Commands` directory to understand the pattern. For executing other Artisan commands, use the `Artisan::call()` facade. The command should be registered in `app/Console/Kernel.php`. Also, note that the command could accept additional arguments to run other steps. The assignee is currently assigned, so coordinate with them before starting.