Skip to content
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

Added my keymap for Rorschach keyboard; minor updates to my Kinesis files. #3734

Merged
merged 5 commits into from Aug 25, 2018
Merged

Added my keymap for Rorschach keyboard; minor updates to my Kinesis files. #3734

merged 5 commits into from Aug 25, 2018

Conversation

tuesdayjohn
Copy link
Contributor

@tuesdayjohn tuesdayjohn commented Aug 24, 2018

Added my keymap for Rorschach keyboard; minor updates to my Kinesis files.

@tuesdayjohn tuesdayjohn changed the title Added keymap for Rorschach Added my keymap for Rorschach keyboard; minor updates to my Kinesis files. Aug 24, 2018
#ifndef CONFIG_USER_H
#define CONFIG_USER_H

#include "config_common.h"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line isn't actually needed.

@@ -0,0 +1,30 @@

#ifndef CONFIG_USER_H
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you could remove the ifndef and define blocks (as well as the endif at the end) and replace it with #pragma once?

SLEEP_LED_ENABLE = no # Breathing sleep LED during USB suspend
TAP_DANCE_ENABLE = yes # Enable Tap Dancing function

ifndef QUANTUM_DIR
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This ifndef block isn't needed, and is a holdover from the old makefile system.

@drashna drashna added the keymap label Aug 25, 2018
@tuesdayjohn
Copy link
Contributor Author

Changes have been added.

@drashna
Copy link
Member

drashna commented Aug 25, 2018

Thanks!

@drashna drashna merged commit 7e8c0a3 into qmk:master Aug 25, 2018
alexey-danilov pushed a commit to alexey-danilov/qmk_firmware that referenced this pull request Sep 6, 2018
…inesis files. (qmk#3734)

* Added keymap for Rorschach

* Rorschach update

* Minor updates to Kinesis files

* Minor fixes

* Minor fixes for Rorschach
dstrelau pushed a commit to dstrelau/qmk_firmware that referenced this pull request Sep 7, 2018
* master: (691 commits)
  mitosis: minor readme improvements (qmk#3762)
  add linebreaks to "Keyboard Maintainer:..." blocks where missing (qmk#3763)
  Keymap: Made changes to buswerks ortho_4x12 keymap (qmk#3757)
  Keyboard: Putting all my ducks in a row: Lightsaver (qmk#3758)
  Keyboard: Make a 1upkeyboards directory for visibility (qmk#3759)
  Helpful functions
  Keymap: initial migration of raffle planck (qmk#3748)
  Keyboard: Duck Jetfire QMK Support (qmk#3752)
  Keymap: initial migration of raffle 1up60rgb (qmk#3747)
  Keymap: PyroL Let's Split keymap + Alpha keymap modifications (qmk#3753)
  Keyboard: Change the default LAYOUT to LAYOUT_60_hhkb for tokyo60 (qmk#3751)
  Keyboard: Tokyo60 Cleanup: QMK Configurator support (qmk#3749)
  Keyboard: add tokyo60 (qmk#2861)
  Keyboard: Added gamepad layout to cospad (qmk#3714)
  Keymap: Added my keymap for Rorschach keyboard; minor updates to my Kinesis files. (qmk#3734)
  Keyboard: readme.md update (qmk#3581)
  Keymap: update to budi's tv44 (qmk#3587)
  Keyboard: introduce new default layout 68_ansi (qmk#3745)
  Keymap: Hold for MO and Tap for TG   (qmk#3742)
  Keyboard: add Wavelet (qmk#3739)
  ...
ChrissiQ pushed a commit to ChrissiQ/qmk_firmware that referenced this pull request Sep 25, 2018
…inesis files. (qmk#3734)

* Added keymap for Rorschach

* Rorschach update

* Minor updates to Kinesis files

* Minor fixes

* Minor fixes for Rorschach
yamad pushed a commit to yamad/qmk_firmware that referenced this pull request Apr 10, 2019
…inesis files. (qmk#3734)

* Added keymap for Rorschach

* Rorschach update

* Minor updates to Kinesis files

* Minor fixes

* Minor fixes for Rorschach
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants