qmk/qmk_firmware

[Feature Request] Exclude external userspace from lint checking

Open

#24.676 aberto em 4 de dez. de 2024

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C (43.867 forks)batch import
enhancementhelp wantedto do

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

Feature Request Type

  • Core functionality
  • Add-on hardware support (eg. audio, RGB, OLED screen, etc.)
  • Alteration (enhancement/optimization) of existing feature(s)
  • New behavior

Description

qmk lint can be run locally before submitting a keyboard PRs to avoid unnecessary churn of commits just to satisfy the the CI lint checker. However, when run locally, external userspace keymaps are included in keymap list, so via[l] keymaps get erroneously flagged as "should not exist" even when they are properly located outside of the qmk_firmware repository. When checking for the presence of those keymaps, it would be ideal if the keymap list did not include keymaps found in the external userspace.

Guia do colaborador

Direção de pesquisa
Examine o script qmk lint para entender como ele coleta os keymaps e modifique a lógica de filtragem para excluir keymaps de diretórios de espaço do usuário externo.
Pilha de tecnologia
c
Domain
backendcli
Tipo Issue
Funcionalidade
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
QMK firmwarekeymap structurepython (for lint script)
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40