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
Adding TRKeyboard TRK1 to QMK #23642
base: develop
Are you sure you want to change the base?
Conversation
keyboards/trkeyboards/trk1/config.h
Outdated
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.
If this file has nothing in it, it can and should be removed.
[2] = LAYOUT( | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______ | ||
), | ||
[3] = LAYOUT( | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______ | ||
) |
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.
Empty layouts should be removed, as all they do is eat into usable firmware space.
[2] = LAYOUT( | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______ | |
), | |
[3] = LAYOUT( | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______ | |
) |
[1] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) }, | ||
[2] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) }, | ||
[3] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) } |
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.
[1] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) }, | |
[2] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) }, | |
[3] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) } | |
[1] = { ENCODER_CCW_CW(_______, _______) }, |
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.
This file can be removed.
[2] = LAYOUT( | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______ | ||
), | ||
[3] = LAYOUT( | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | ||
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______ | ||
) |
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.
[2] = LAYOUT( | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______ | |
), | |
[3] = LAYOUT( | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, _______, | |
_______, _______, _______, _______, _______, _______, _______, _______, _______, _______ | |
) |
[1] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) }, | ||
[2] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) }, | ||
[3] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) } |
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.
[1] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) }, | |
[2] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) }, | |
[3] = { ENCODER_CCW_CW(KC_VOLD, KC_VOLU) } | |
[1] = { ENCODER_CCW_CW(_______, _______) } |
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.
This file should be removed.
keyboards/trkeyboards/trk1/readme.md
Outdated
@@ -0,0 +1,30 @@ | |||
# Tvrd Rad Keyboards TRK1 | |||
|
|||
![TRK1](https://www.trkeyboards.com/upload/products/gallery/1663968347286_raaw8387.jpg) |
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.
Please rehost the image on a dedicated image host, such as Imgur (with an account)
No offense, but shops are ephemeral, and regularly vanish.
keyboards/trkeyboards/trk1/readme.md
Outdated
|
||
make trkeyboards/trk1:default | ||
|
||
# or, if you'd like a VIA-enabled keymap: |
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.
# or, if you'd like a VIA-enabled keymap: | |
Or, if you'd like a VIA-enabled keymap: |
keyboards/trkeyboards/trk1/readme.md
Outdated
|
||
make trkeyboards/trk1:default:flash | ||
|
||
# or, if you'd like a VIA-enabled keymap: |
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.
# or, if you'd like a VIA-enabled keymap: | |
Or, if you'd like a VIA-enabled keymap: |
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.
Since you're targeting the develop
branch, the rules.mk
file should be removed, and the info.json
file should be renamed to keyboard.json
.
keyboards/trkeyboards/trk1/readme.md
Outdated
make trkeyboards/trk1:default | ||
|
||
Or, if you'd like a VIA-enabled keymap: | ||
|
||
make trkeyboards/trk1:via | ||
|
||
Flashing example for this keyboard: | ||
|
||
make trkeyboards/trk1:default:flash | ||
|
||
Or, if you'd like a VIA-enabled keymap: | ||
|
||
make trkeyboards/trk1:via:flash |
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.
make trkeyboards/trk1:default | |
Or, if you'd like a VIA-enabled keymap: | |
make trkeyboards/trk1:via | |
Flashing example for this keyboard: | |
make trkeyboards/trk1:default:flash | |
Or, if you'd like a VIA-enabled keymap: | |
make trkeyboards/trk1:via:flash | |
make trkeyboards/trk1:default | |
Flashing example for this keyboard: | |
make trkeyboards/trk1:default:flash |
Description
Adding TRKeyboard TRK1 to QMK
Types of Changes
Checklist