-
-
Notifications
You must be signed in to change notification settings - Fork 39.8k
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
Update Keychron Q4 #21996
Update Keychron Q4 #21996
Conversation
for |
…ychron-q4-update
Can you elaborate more on that? |
No need. It uses the parent folder's keymaps, if they exist. As long as the layout macro is correct and supported, it will compile using those higher level keymaps. You can see this in the QMK CI logs, actually. Also, diff makes it look... wonky. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add an entry to data/mapping/keyboard_aliases.hjson for the name change (since the old name won't compile on the new version)
Description
Superseeds #21406, but this will only cover one board from each series update of PRs.
Types of Changes
Issues Fixed or Closed by This PR
Checklist