pnp/cli-microsoft365

Centralise default config values

Aperta

#6186 aperta il 27 lug 2024

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)TypeScript (399 fork)auto 404
enhancementhelp wantedkeep-open

Metriche repository

Star
 (1390 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

In a lot of commands, we retrieve config values as follows:

cli.getSettingWithDefaultValue<boolean>(settingsNames.autoOpenLinksInBrowser, false)

I don't think it's a good approach to define the default value in the command itself. A better approach would be that we centralize all default values and provide a new function where the user should only specify the config variable. E.g.:

cli.getSetting<boolean>(settingsNames.autoOpenLinksInBrowser)

Guida contributor

Direzione di ricerca
Cerca nel codebase tutte le occorrenze di 'getSettingWithDefaultValue' ed elenca i nomi delle impostazioni e i loro valori predefiniti. Quindi proponi una mappatura centralizzata o una nuova funzione che recuperi automaticamente il valore predefinito.
Tech stack
typescript
Dominio
clidevtools
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
TypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80