nunomaduro/phpinsights

[Laravel] Allow to set command parameters from config file

Open

#376 aperta il 10 mar 2020

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)PHP (308 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (5092 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Q A
Bug report? no
Feature request? yes
Library version 1.13.0

Related with https://github.com/nunomaduro/phpinsights/issues/23 will be a great idea to allow set all commands parameters from config/insights.php when is used with php artisan insights.

Maybe something like:

'parameters' => '--format=json -d memory_limit=2000M -v',

or

'parameters' => [
    '--format=json',
    '-d memory_limit=2000M',
    '-v',
],

Parameters from command line must overwrite config parameters:

php artisan insights --format=html

And also an option to avoid use config will be great:

php artisan insights --no-config-parameters

Thanks!

Guida contributor

Direzione di ricerca
Investiga come il pacchetto legge attualmente il suo file di configurazione e passa i parametri al comando sottostante. Dovresti esaminare la struttura del file config/insights.php, la classe del comando che gestisce artisan insights e come fonde gli argomenti CLI con le opzioni predefinite. Implementare questa funzionalità richiede la modifica del comando per accettare una chiave di configurazione aggiuntiva (ad esempio 'parameters') e unirla con gli argomenti CLI, assicurandoti che gli argomenti CLI abbiano la precedenza. Inoltre, aggiungi un flag no config parameters per saltare la lettura dei parametri di configurazione.
Tech stack
php
Dominio
cli
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PHPLaravel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60