yarnpkg/yarn

Inconsistent config strings

Open

#4.496 geöffnet am 18. Sept. 2017

Auf GitHub ansehen
 (9 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.731 Forks)batch import
cat-featuregood first issuehelp wantedtriaged

Repository-Metriken

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

Beschreibung

Continuation from #4457

Do you want to request a feature or report a bug? Potential bug.

What is the current behavior? Yarn config has different formatting for object keys that are computed / set vs defaults.

I'm not sure I understand if this is by design or not. If so, what is the use case?

Initial registry configuration

What is the expected behavior? Keys should all be wrapped in single quotes or DEFAULTS should follow same formatting as registry, lastUpdateCheck, and user configured values do

Please mention your node.js, yarn and operating system version. Yarn: 1.0.2 Node: 8.1.4

Happy to submit a PR if the current functionality isn't as expected.

Contributor Guide

Research-Richtung
Untersuche den Yarn Registry Code in src/registries/yarn registry.js um Zeile 15, um zu verstehen, wie Standardkonfigurationsschlüssel definiert werden, und vergleiche sie mit der Ausgabeformatierung, um eine konsistente Verwendung von einfachen Anführungszeichen sicherzustellen.
Tech Stack
javascriptnodejs
Domain
cli
Issue Type
Fehler
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.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80