qmk/qmk_firmware

[Bug] info.json in subfolder not overridding info.json in top-level folder

Open

#19.256 geöffnet am 7. Dez. 2022

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C (43.867 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (20.368 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 20T 9h) (27 gemergte PRs in 30 T)

Beschreibung

Describe the Bug

If a feature is set in a top-level keyboard folder's info.json (ex. keebio/sinc/info.json), it can't be overridden by a subfolder's info.json (ex. keebio/sinc/rev2/info.json). Overridding a feature using rules.mk (ex. keebio/sinc/rev2/rules.mk) works fine.

Keyboard Used

qmk info -f json -kb keebio/sinc/rev2 -km via | head -n50

Link to product page (if applicable)

No response

Operating System

No response

qmk doctor Output

No response

Is AutoHotKey / Karabiner installed

  • AutoHotKey (Windows)
  • Karabiner (macOS)

Other keyboard-related software installed

No response

Additional Context

No response

Contributor Guide

Research-Richtung
Untersuche, wie das Build System von QMK die info.json Dateien aus dem Hauptverzeichnis der Tastatur und den Unterordnern für Revisionen zusammenführt. Suche nach dem Code, der das Laden und Zusammenführen von JSON verarbeitet, möglicherweise in `lib/python/qmk/info.py` oder ähnlich. Prüfe, ob eine rekursive Merge Logik fehlt oder ob es Probleme mit der Reihenfolgenabhängigkeit gibt.
Tech Stack
c
Domain
Keine
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
CQMK firmware build system
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50