Miserlou/Zappa

Check Config Files for Allowed Values

Open

#1.165 geöffnet am 11. Okt. 2017

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (1.286 Forks)batch import
hacktoberfesthelp wanted

Repository-Metriken

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

Beschreibung

Context

Was not seeing my remote environment variables because my config value was "remote_evn". Mistake on my part, but it would be nice if zappa checked config files to ensure that values being set were even values used by zappa.

Expected Behavior

When a user gives a config value not used by Zappa, it tells the user and suggests that it might be a typo. Just lowers the learning curve slightly and improves QOL.

Actual Behavior

Users are allowed to wallow in their mistakes

Possible Fix

Keep a white list of config keys and test against those.

Contributor Guide

Research-Richtung
Das Problem schlägt vor, eine Whitelist gültiger Konfigurationsschlüssel in der Konfigurationsvalidierung von Zappa hinzuzufügen. Untersuchen Sie Dateien wie 'zappa/cli.py' oder 'zappa/settings.py', in denen das Konfigurationsparsing stattfindet. Die Maintainer haben keine zusätzlichen Anleitungen bereitgestellt; der vorgeschlagene Fix besteht darin, eine Menge erlaubter Schlüssel zu pflegen und diese gegen die vom Benutzer bereitgestellten Schlüssel zu prüfen, wobei eine Warnung oder ein Fehler für unbekannte Schlüssel ausgegeben wird.
Tech Stack
python
Domain
backend
Issue Type
Funktion
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.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85