pnp/cli-microsoft365

Centralise default config values

Offen

#6.186 geöffnet am 27.07.2024

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (399 Forks)auto 404
enhancementhelp wantedkeep-open

Repository-Metriken

Stars
 (1.390 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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)

Contributor Guide

Research-Richtung
Durchsuche die Codebasis nach allen Vorkommen von 'getSettingWithDefaultValue' und liste die Einstellungsnamen und deren Standardwerte auf. Schlage dann eine zentrale Zuordnung oder eine neue Funktion vor, die den Standardwert automatisch abruft.
Tech Stack
typescript
Domain
clidevtools
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80