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

E6V2 LE HHKB love #4079

Merged
merged 2 commits into from
Oct 5, 2018
Merged

E6V2 LE HHKB love #4079

merged 2 commits into from
Oct 5, 2018

Conversation

johu
Copy link
Contributor

@johu johu commented Oct 5, 2018

  • HHKB layout support
  • my layout

@drashna
Copy link
Member

drashna commented Oct 5, 2018

@noroadsleft This look good to you?

keyboards/e6v2/le/keymaps/johu/keymap.c Outdated Show resolved Hide resolved
$ make e6v2/le:johu:dfu
```

See [build environment setup] then the [make instructions] for more information.
Copy link
Member

Choose a reason for hiding this comment

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

Please change this line to:

See the [build environment setup](https://docs.qmk.fm/#/getting_started_build_tools) and the [make instructions](https://docs.qmk.fm/#/getting_started_make_guide) for more information. Brand new to QMK? Start with our [Complete Newbs Guide](https://docs.qmk.fm/#/newbs).

and then delete Lines 35 and 36.

Copy link
Contributor Author

@johu johu Oct 5, 2018

Choose a reason for hiding this comment

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

I will add the the last sentence, but I prefer the links at the bottom as the markdown is way more readable in source.

Copy link
Contributor

@mechmerlin mechmerlin Oct 5, 2018

Choose a reason for hiding this comment

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

This information should also be in the keyboard's readme, not just in a keymap readme.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This information should also be in the keyboard's readme, not just in a keymap readme.

Done

@johu johu force-pushed the e6v2 branch 2 times, most recently from 8ad99cf to cfe3a70 Compare October 5, 2018 21:10
@drashna
Copy link
Member

drashna commented Oct 5, 2018

Awesome!

@drashna drashna merged commit 54b61b5 into qmk:master Oct 5, 2018
@johu
Copy link
Contributor Author

johu commented Oct 5, 2018

Squashing those commits was not good imho, as they are logical separated.

@drashna
Copy link
Member

drashna commented Oct 5, 2018

You're right. Sorry.

Though, generally, this should have been two separate PRs, one for the keyboard, and one for the keymap.

This is "my bad". If you would like, we can revert it and then re-submit it, but it should be okay as is.

@drashna
Copy link
Member

drashna commented Oct 5, 2018

https://docs.qmk.fm/#/contributing?id=keymaps
https://docs.qmk.fm/#/contributing?id=keyboards

So, definitely my fault. WIll try to do better in the future.

akatrevorjay added a commit to akatrevorjay/qmk_firmware that referenced this pull request Oct 8, 2018
* 'master' of git://github.com/qmk/qmk_firmware: (302 commits)
  Keymap: Added Planck keymap: jcorrado (qmk#4083)
  Keyboard: Helix five_rows keymap reduced size (qmk#4082)
  E6V2 LE HHKB love (qmk#4079)
  Keymap: Add french iso layout for dz60 (thomasviaud) (qmk#4080)
  ARM SWD guide
  Keyboard: set SAFE_RANGE (qmk#4078)
  Keyboard: DZ60: add Plate B layouts (qmk#4076)
  rgb_matrix: continue calling rgb_matrix_indicators() when toggled off
  Keymap: Added personal userspace and keymaps. (qmk#4073)
  Add mousekey_send to (un)register_code
  rn42 compilation error fix (qmk#4050)
  process_ucis: Add a callback for success (qmk#4067)
  Keyboard: add info.json for qmk configurator (qmk#4063)
  Keyboard: Adding Split Space support for Mechmini 1 (qmk#4052)
  Keyboard: Added dactyl-manuform 4x6 and keymap. (qmk#4058)
  Keyboard: Add handwired/pteron config and keymaps (qmk#4060)
  Keyboard: Added split backspace layout for the b.mini (qmk#4051)
  Keyboard: Add namecard keyboard (qmk#4044)
  handwired/ibm122m: refactor and Configurator support (qmk#4054)
  Keyboard: Added JN68M keyboard to handwired (qmk#4053)
  ...
zer09 pushed a commit to zer09/qmk_firmware that referenced this pull request Oct 13, 2018
* Keyboard: E6V2 LE HHKB layout support

* Keymap: Add johu's E6V2 LE HHKB layout
rseymour pushed a commit to rseymour/qmk_firmware that referenced this pull request Mar 13, 2019
* Keyboard: E6V2 LE HHKB layout support

* Keymap: Add johu's E6V2 LE HHKB layout
yamad pushed a commit to yamad/qmk_firmware that referenced this pull request Apr 10, 2019
* Keyboard: E6V2 LE HHKB layout support

* Keymap: Add johu's E6V2 LE HHKB layout
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants