qmk/qmk_firmware

[Feature Request] Exclude external userspace from lint checking

Open

#24.676 aperta il 4 dic 2024

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C (43.867 fork)batch import
enhancementhelp wantedto do

Metriche repository

Star
 (20.368 star)
Metriche merge PR
 (Merge medio 20g 9h) (27 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina lo script qmk lint per capire come raccoglie le mappe dei tasti e modifica la logica di filtro per escludere le mappe provenienti dalle directory utente esterne.
Tech stack
c
Dominio
backendcli
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
QMK firmwarekeymap structurepython (for lint script)
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40