-
-
Notifications
You must be signed in to change notification settings - Fork 39.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Durgod K310/K320 Refactor #18224
Durgod K310/K320 Refactor #18224
Conversation
|
Move `kuenhlee` keymap from `durgod/k3x0/k320/keymaps/` to `durgod/k320/keymaps/`.
Relocate/duplicate the source files in `durgod/k3x0/` to `durgod/k310/(base/)` and `durgod/k320/(base/)` as needed.
Remove the existing `LAYOUT_all` macro (it's for the K310 fullsize) and rename `LAYOUT_tkl` to `LAYOUT_all`.
Update keyboard metadata list formatting.
I don't understand why the lint check fails on noroadsleft ~/qmk_firmware/keyboards/durgod (rf/durgod)
$ git log -1 --oneline
d73821aea3 (HEAD -> rf/durgod, origin/rf/durgod) fix broken links in keyboard readmes
noroadsleft ~/qmk_firmware/keyboards/durgod (rf/durgod)
$ qmk compile -e SILENT=true -kb durgod/k320/base -km default
Ψ Compiling keymap with make --jobs=4 --output-sync=target SILENT=true durgod/k320/base:default
Making durgod/k320/base with keymap default [OK] |
That was exactly it. I messed up a |
PR NotesThis PR consolidates the build options for the Durgod K310 and K320 keyboards as follows:
As such, keymaps have been relocated as follows:
@moults31, your keymap remains present in |
Description
Simplifies the structure of the Durgod K310 and K320 codebases, at the cost of some code duplication.
Types of Changes
Checklist