qmk/qmk_firmware

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

Open

#19.256 aberto em 7 de dez. de 2022

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C (43.867 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (20.368 stars)
Métricas de merge de PR
 (Mesclagem média 20d 9h) (27 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Investigue como o sistema de build do QMK mescla os arquivos info.json do diretório raiz do teclado e das subpastas de revisão. Procure o código que lida com o carregamento e mesclagem de JSON, possivelmente em `lib/python/qmk/info.py` ou similar. Verifique se falta lógica de mesclagem recursiva ou se há problemas de dependência de ordem.
Pilha de tecnologia
c
Domain
None
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
CQMK firmware build system
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50