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

add keyboard support for duckboard #10318

Merged
merged 24 commits into from
Sep 25, 2020
Merged

add keyboard support for duckboard #10318

merged 24 commits into from
Sep 25, 2020

Conversation

doodboard
Copy link
Contributor

@doodboard doodboard commented Sep 15, 2020

Adding duckboard support to QMK

Description

Adding support for Duckboard to QMK.
Added new folder "duckboard" under keyboard.
Added config.h / kb.c / kb.h / Makefile / rules.mk / Keymap.c to relevant directory.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • [x ] Keyboard (addition or update)
  • [x ] 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).

@doodboard doodboard changed the title Duckboard add support for duckboard Sep 16, 2020
@doodboard doodboard changed the title add support for duckboard add keyboard support for duckboard Sep 16, 2020
keyboards/doodboard/duckboard/config.h Outdated Show resolved Hide resolved
keyboards/doodboard/duckboard/config.h Outdated Show resolved Hide resolved
keyboards/doodboard/duckboard/kb.c Outdated Show resolved Hide resolved
keyboards/doodboard/duckboard/kb.h Outdated Show resolved Hide resolved
keyboards/doodboard/duckboard/kb.h Outdated Show resolved Hide resolved
keyboards/doodboard/duckboard/keymaps/default/keymap.c Outdated Show resolved Hide resolved
keyboards/doodboard/duckboard/rules.mk Outdated Show resolved Hide resolved
@drashna drashna requested a review from a team September 21, 2020 00:05
@drashna drashna added the keymap label Sep 21, 2020
doodboard and others added 10 commits September 21, 2020 09:48
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Update code per review
folder name errod
@doodboard
Copy link
Contributor Author

Sorry for the mistakes. This is my first time contributing and there is a lot to learn, clearly. Thank you in advance for the time and effort!

Copy link
Member

@fauxpark fauxpark left a comment

Choose a reason for hiding this comment

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

This board is also missing an info.json.

keyboards/doodboard/duckboard/config.h Outdated Show resolved Hide resolved
keyboards/doodboard/duckboard/config.h Outdated Show resolved Hide resolved
keyboards/doodboard/duckboard/rules.mk Outdated Show resolved Hide resolved
keyboards/doodboard/duckboard/rules.mk Outdated Show resolved Hide resolved
doodboard and others added 6 commits September 21, 2020 16:47
@doodboard
Copy link
Contributor Author

comments applied, and I added info.json!

@doodboard
Copy link
Contributor Author

the firmware compiles fine on my end, I'm not sure why it's failing the checks

Co-authored-by: Ryan <fauxpark@gmail.com>
keyboards/doodboard/duckboard/info.json Outdated Show resolved Hide resolved
doodboard and others added 2 commits September 21, 2020 17:32
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
@drashna drashna requested a review from a team September 22, 2020 05:53
Co-authored-by: James Young <18669334+noroadsleft@users.noreply.github.com>
@doodboard
Copy link
Contributor Author

doodboard commented Sep 24, 2020

Thank you guys for the reviews! Is there something else I need to do at this point?

@noroadsleft
Copy link
Member

Thank you guys for the reviews! Is there something else I need to do at this point?

Nope; looks good, thanks!

@noroadsleft noroadsleft merged commit 4a83021 into qmk:master Sep 25, 2020
tomohisa pushed a commit to tomohisa/qmk_firmware that referenced this pull request Sep 28, 2020
* 'master' of https://github.com/qmk/qmk_firmware: (3643 commits)
  Swapparoo of bottom row keys (qmk#10277)
  [Keyboard] Add PloopyCo devices (qmk#7935)
  Keyboard update: SL40 (qmk#10445)
  [keyboard] Add SP-111 support (qmk#10193)
  Add logic for AT90USBxx7 where needed (qmk#10203)
  Trifecta Keymap
  [Docs] Wording & formatting changes in the Tapping Force Hold section of the Tap Hold page + removed trailing whitespaces (qmk#10391)
  `setrgb()`: Use arrow operator (qmk#10451)
  [Keyboard] add Percent Canoe Gen2 (qmk#10344)
  [Keyboard] add hannah65 by Team Mechlovin (qmk#10284)
  Fix Belgian sendstring properly (qmk#10444)
  Add VIA support for lazydesigners/the30 (qmk#10374)
  [Keyboard] add duckboard by doodboard (qmk#10318)
  Fix Belgian sendstring file (qmk#10443)
  [Keyboard] added Bolsa65 keyboard by FJLabs (qmk#10394)
  CLI/Doctor: Print QMK_HOME (qmk#10398)
  Add Wyvern Keyboard (qmk#10378)
  Adding Bear 65 Ergo Keyboard To QMK (qmk#10384)
  VIA Support: Gray Studio HB85 (qmk#10329)
  format code according to conventions [skip ci]
  ...
rgoulter pushed a commit to rgoulter/qmk_firmware that referenced this pull request Oct 4, 2020
* Adding duckboard to QMK

Adding duckboard support to QMK

* added duckboard under doodboard

* changed debouncing delay to debounce

* Update keyboards/doodboard/duckboard/config.h

* Update keyboards/doodboard/duckboard/config.h

* Update keyboards/doodboard/duckboard/kb.h

* Update keyboards/doodboard/duckboard/keymaps/default/keymap.c

* Update keyboards/doodboard/duckboard/rules.mk

* Update keyboards/doodboard/duckboard/kb.h

* Update keyboards/doodboard/duckboard/kb.c

* Update code per review

Update code per review

* folder name errod

folder name errod

* Update keyboards/doodboard/duckboard/config.h

* Update keyboards/doodboard/duckboard/rules.mk

* Update keyboards/doodboard/duckboard/rules.mk

* Update keyboards/doodboard/duckboard/config.h

* Create info.json

* Update keyboards/doodboard/duckboard/duckboard.h

* Update keyboards/doodboard/duckboard/info.json

* Update keyboards/doodboard/duckboard/info.json

* Update keyboards/doodboard/duckboard/duckboard.h

* Update keyboards/doodboard/duckboard/readme.md
kjganz pushed a commit to kjganz/qmk_firmware that referenced this pull request Oct 28, 2020
* Adding duckboard to QMK

Adding duckboard support to QMK

* added duckboard under doodboard

* changed debouncing delay to debounce

* Update keyboards/doodboard/duckboard/config.h

* Update keyboards/doodboard/duckboard/config.h

* Update keyboards/doodboard/duckboard/kb.h

* Update keyboards/doodboard/duckboard/keymaps/default/keymap.c

* Update keyboards/doodboard/duckboard/rules.mk

* Update keyboards/doodboard/duckboard/kb.h

* Update keyboards/doodboard/duckboard/kb.c

* Update code per review

Update code per review

* folder name errod

folder name errod

* Update keyboards/doodboard/duckboard/config.h

* Update keyboards/doodboard/duckboard/rules.mk

* Update keyboards/doodboard/duckboard/rules.mk

* Update keyboards/doodboard/duckboard/config.h

* Create info.json

* Update keyboards/doodboard/duckboard/duckboard.h

* Update keyboards/doodboard/duckboard/info.json

* Update keyboards/doodboard/duckboard/info.json

* Update keyboards/doodboard/duckboard/duckboard.h

* Update keyboards/doodboard/duckboard/readme.md
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.

4 participants