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

[Keyboard] Add Borderland #15501

Merged
merged 8 commits into from
Dec 18, 2021
Merged

[Keyboard] Add Borderland #15501

merged 8 commits into from
Dec 18, 2021

Conversation

kiwikey
Copy link

@kiwikey kiwikey commented Dec 17, 2021

Description

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@github-actions github-actions bot added keyboard keymap via Adds via keymap and/or updates keyboard for via support labels Dec 17, 2021
keyboards/kiwikey/borderland/config.h Outdated Show resolved Hide resolved
keyboards/kiwikey/borderland/keymaps/default/keymap.c Outdated Show resolved Hide resolved
keyboards/kiwikey/borderland/keymaps/default/encoder.c Outdated Show resolved Hide resolved
keyboards/kiwikey/borderland/keymaps/via/encoder.c Outdated Show resolved Hide resolved
keyboards/kiwikey/borderland/keymaps/via/keymap.c Outdated Show resolved Hide resolved
keyboards/kiwikey/borderland/borderland.c Show resolved Hide resolved
kiwikey and others added 3 commits December 17, 2021 14:48
Apply changes, follow suggestions from Drashna's review:
+ Moved code in "encoder.c" into "borderland.c"
+ Removed definition of "encoder.c" in all keymaps
+ Removed two "encoder.c" files
+ Explicitly defined all RGB Lighting animations, not using "#define RGBLIGHT_ANIMATIONS"

Co-authored-by: Drashna Jaelre <[email protected]>
@kiwikey
Copy link
Author

kiwikey commented Dec 17, 2021

Hi @drashna, thank you for reviewing the codes. I have updated and commited changes as your suggestion.
I also re-built firmwares in my local PC, and tested with PCB, they all works.

Please help to re-check again and merge my PR if there is no issues left.

P.s: I am not so familiar with Git, hope all gonna be fine. Please let me know if there is need of any correction.

@drashna drashna requested a review from a team December 17, 2021 16:03
@drashna drashna merged commit 4b9ba0d into qmk:master Dec 18, 2021
@drashna
Copy link
Member

drashna commented Dec 18, 2021

Thanks!

@kiwikey
Copy link
Author

kiwikey commented Dec 18, 2021

Thank you guys!!! Happy weekend.

@kiwikey kiwikey deleted the dev_kiwikey branch December 18, 2021 03:26
mattgauf added a commit to mattgauf/qmk_firmware that referenced this pull request Dec 20, 2021
* master: (581 commits)
  Add Superlyra keyboard + group with Lyra under new directory (qmk#14955)
  MechBrewery MB65S Refactor (qmk#15515)
  [Keyboard] Add k34 keyboard (qmk#15250)
  [Keyboard] update new keyboard zoom65_lite (qmk#15218)
  [Keyboard] Add Borderland (qmk#15501)
  [Keyboard] Nasu (qmk#15468)
  [Keyboard] Remove locking support for Sweep (qmk#15494)
  [Keyboard] Remove locking support from Cradio (qmk#15495)
  [Keyboard] Remove locking support for Macro3 (qmk#15496)
  [Keyboard] Add WestM (qmk#15459)
  Fix inverted gt and lt send_string mapping in canadian multilingual (qmk#15487)
  [Keyboard] removed extra layout from jorne.h (qmk#15491)
  [Keyboard] Add Jorne (qmk#15461)
  Fix some wrong keycode macros in locale mapping files (qmk#15488)
  [Core] Fix Sensor driver code (qmk#15484)
  [Keymap] Unicode and Pointing Device and Autocorect for drashna keymaps (qmk#15415)
  [Keyboard] Adelie Firmware (qmk#13265)
  [Keyboard] Add Elite-C DFU bootloader commands for Sweep (qmk#15449)
  [Keyboard] W1-AT addition (qmk#15467)
  [Keyboard] Add swap hands support for Technik_O (qmk#15477)
  ...
epiciskandar pushed a commit to epiciskandar/qmk_firmware that referenced this pull request Dec 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keyboard keymap via Adds via keymap and/or updates keyboard for via support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants