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] Added bradpad support #12912

Merged
merged 4 commits into from
May 16, 2021
Merged

[Keyboard] Added bradpad support #12912

merged 4 commits into from
May 16, 2021

Conversation

wxyangf
Copy link

@wxyangf wxyangf commented May 15, 2021

Description

Added support for the bradpad an upcoming stocked macropad

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).

Extra Info:

see: #12907 for commit history, chose to rebranch and compress because of some submodule weirdness.

@github-actions github-actions bot added keyboard keymap via Adds via keymap and/or updates keyboard for via support labels May 15, 2021
@wxyangf wxyangf mentioned this pull request May 15, 2021
6 tasks
keyboards/tempo_turtle/bradpad/config.h Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/config.h Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/config.h Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/config.h Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/keymaps/4brad/config.h Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/keymaps/midi_demo/config.h Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/keymaps/4brad/keymap.c Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/keymaps/4brad/keymap.c Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/keymaps/midi_demo/keymap.c Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/keymaps/via/keymap.c Outdated Show resolved Hide resolved
@wxyangf
Copy link
Author

wxyangf commented May 16, 2021

@drashna do you have any idea why it keeps trying to pull in whatever's going on with chibiOS and how to avoid it? I haven't touched the submodules at all but it keeps sneaking into my commits.

Last time I fixed it by just creating a new branch, but this seems like it just wants to keep creeping back.

@wxyangf wxyangf marked this pull request as draft May 16, 2021 03:43
@wxyangf
Copy link
Author

wxyangf commented May 16, 2021

For reference how I resolved the chibiOS stuff is as follows:

  1. Added the chibios directories to .git/info/exclude
  2. flagged the chibios directories as "assume unchanged" with git.

I might make a pr for a change to the .gitmodules the future to set the chibios as "ignore=dirty" because I know that I'm not the only one who has had to deal with that weirdness.

@wxyangf wxyangf marked this pull request as ready for review May 16, 2021 04:07
keyboards/tempo_turtle/bradpad/readme.md Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/keymaps/midi_demo/keymap.c Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/keymaps/default/keymap.c Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/keymaps/4brad/keymap.c Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/readme.md Outdated Show resolved Hide resolved
keyboards/tempo_turtle/bradpad/rules.mk Outdated Show resolved Hide resolved
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
@drashna
Copy link
Member

drashna commented May 16, 2021

Thanks!

@drashna drashna merged commit 6d1f059 into qmk:master May 16, 2021
@wxyangf
Copy link
Author

wxyangf commented May 16, 2021

Thanks for the review guys!

HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Jul 11, 2021
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
nhongooi pushed a commit to nhongooi/qmk_firmware that referenced this pull request Dec 5, 2021
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
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