Differences between config_site.py and config_main.py

Offen
#370 7 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
48/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
docker, python
Bereich
devops

Rechercherichtung

Beginne mit dem Vergleich von docker/config_site.py und GeoHealthCheck/config_main.py und konzentriere dich dabei auf SQLALCHEMY_TRACK_MODIFICATIONS, GHC_METADATA_CACHE_SECS, GHC_PROBE_DEFAULTS, GHC_PLUGINS und GHC_USER_PLUGINS. Erledigt ist die Aufgabe, wenn die Docker-Konfiguration die fehlenden Variablen bereitstellt und die Plugin-Werte konsistent mit der Hauptkonfiguration verarbeitet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

Hi, I've noticed a couple of differences between the files docker/config_site.py and GeoHealthCheck/config_main.py. The latter is containing the variables SQLALCHEMY_TRACK_MODIFICATIONS, GHC_METADATA_CACHE_SECS and GHC_PROBE_DEFAULTS, which are missing in the former. It might be a good idea to add them to the Docker config as well.

As an additional note, config_main.py is using arrays for the variables GHC_PLUGINS and GHC_USER_PLUGINS, but when reading them as env. vars in config_site.py, the values will be strings. That doesn't seem correct to me.

Vorherrschende Sprache
Python
Sterne
94
Forks
77
Ø Merge
27 Min.
Gemergte PRs (30 T.)
1

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus geopython/GeoHealthCheck

Alle Issues in geopython/GeoHealthCheck

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.