jennetters iris keymap #3488
Merged
jennetters iris keymap #3488
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
drashna
requested changes
Jul 25, 2018
| along with this program. If not, see <http://www.gnu.org/licenses/>. | ||
| */ | ||
|
|
||
| #ifndef CONFIG_USER_H |
There was a problem hiding this comment.
Could you replace the boilerplate (the ifndef and define, and endif at the bottom) with just #pragma once ?
|
|
||
| #define TAPPING_TERM 150 | ||
|
|
||
| #include "config_common.h" |
There was a problem hiding this comment.
[Optional] This line isn't needed, at all
| @@ -0,0 +1,209 @@ | |||
| #include "iris.h" | |||
There was a problem hiding this comment.
Could you replace all of these includes with just #include QMK_KEYBOARD_H?
| RGBLIGHT_ENABLE = yes | ||
| BACKLIGHT_ENABLE = yes | ||
|
|
||
| ifndef QUANTUM_DIR |
There was a problem hiding this comment.
This block isn't needed, and should be removed.
drashna
approved these changes
Jul 26, 2018
|
Awesome, thanks! |
|
Thanks for your help! |
mattfluder
pushed a commit
to mattfluder/qmk_firmware
that referenced
this pull request
Jul 30, 2018
* jennetters iris keymap, re-write for improved readability. * Update rules/config/keymap to follow correct makefile structure
alexey-danilov
pushed a commit
to alexey-danilov/qmk_firmware
that referenced
this pull request
Aug 3, 2018
* jennetters iris keymap, re-write for improved readability. * Update rules/config/keymap to follow correct makefile structure
tenderlove
added a commit
to tenderlove/qmk_firmware
that referenced
this pull request
Aug 5, 2018
* master: (184 commits) Docs: Clarify "right" -> "right-handed" (qmk#3507) Docs: Fixed the link to the advanced keycodes wiki (qmk#3505) Keyboard: Update KEYMAP to LAYOUT for ckeys obelus (qmk#3495) Keyboard: Hide JC65 directory in QMK Configurator (qmk#3491) Keymap: jennetters iris keymap (qmk#3488) Keymap: jennetters fourier split 40% keymap (qmk#3487) Keymap: Add LAYOUT_planck_grid to planck/dshields keymap. (qmk#3484) Keyboard: Add HelixPico keyboard (qmk#3502) Keymap: Ergodox Dvorak 42 key layout changes (qmk#3500) Keyboard: E6V2 Layout Fix (qmk#3490) Keymap: UT47:nordic - A couple of tweaks (qmk#3489) Keyboard: E6V2 LE Improvements (qmk#3486) redox: convert to use SPLIT_KEYBOARD feature (qmk#3478) Keymap: Bocaj Ergodox Updates (qmk#3364) keymap: Update khord let's split keymap and add chocopad keymap (qmk#3482) Keymap: Add smt keymap for TKC1800 (qmk#3481) Fix for the Split Common I2C code's SCL_CLOCK issue (qmk#3474) keymap: UT47:nordic - fixed typo (qmk#3477) Keyboard: Add QC60 (qmk#3472) Keyboard: Base for the nicekey keyboard (qmk#3475) ...
filoxo
pushed a commit
to filoxo/qmk_firmware
that referenced
this pull request
Aug 27, 2018
* jennetters iris keymap, re-write for improved readability. * Update rules/config/keymap to follow correct makefile structure
ChrissiQ
pushed a commit
to ChrissiQ/qmk_firmware
that referenced
this pull request
Sep 25, 2018
* jennetters iris keymap, re-write for improved readability. * Update rules/config/keymap to follow correct makefile structure
yamad
pushed a commit
to yamad/qmk_firmware
that referenced
this pull request
Apr 10, 2019
* jennetters iris keymap, re-write for improved readability. * Update rules/config/keymap to follow correct makefile structure
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Re-write for improved readability.