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 hbcp #7101

Merged
merged 36 commits into from
Oct 31, 2019
Merged

Add hbcp #7101

merged 36 commits into from
Oct 31, 2019

Conversation

hineybush
Copy link
Contributor

@hineybush hineybush commented Oct 21, 2019

Description

Add initial hbcp files

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


}

#ifdef RGBLIGHT_ENABLE
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto for this, it's duplicate if you move it to keyboard.c

@yanfali
Copy link
Contributor

yanfali commented Oct 21, 2019

Generating info.json is pretty easy using raw kle. Just copy it into qmk.fm/converter once per layout you want to support and copy it to appropriate place. @noroadsleft can help out if you need it

Copy link
Contributor

@yanfali yanfali left a comment

Choose a reason for hiding this comment

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

Looks good to me. @fauxpark you probably have some feedback on rules.mk

keyboards/hineybush/hbcp/hbcp.c Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/hbcp.c Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/hbcp.c Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/hbcp.c Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/readme.md Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/rules.mk Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/rules.mk Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/rules.mk Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/rules.mk Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/rules.mk Outdated Show resolved Hide resolved
hineybush and others added 13 commits October 28, 2019 13:29
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
keyboards/hineybush/hbcp/rules.mk Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/rules.mk Show resolved Hide resolved
keyboards/hineybush/hbcp/keymaps/hiney/keymap.c Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/keymaps/hiney/keymap.c Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/keymaps/hiney/keymap.c Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/rules.mk Show resolved Hide resolved
keyboards/hineybush/hbcp/hbcp.c Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/info.json Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/info.json Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/keymaps/hiney/keymap.c Outdated Show resolved Hide resolved
keyboards/hineybush/hbcp/keymaps/hiney/readme.md Outdated Show resolved Hide resolved
hineybush and others added 7 commits October 30, 2019 21:09
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: fauxpark <fauxpark@gmail.com>
Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>
Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>
Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>
keyboards/hineybush/hbcp/info.json Outdated Show resolved Hide resolved
Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>
Copy link
Member

@drashna drashna left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Member

@noroadsleft noroadsleft left a comment

Choose a reason for hiding this comment

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

Spoke a bit too soon.

You deleted your color definitions from the hiney/keymap.c, but they never made it to hiney/config.h. The hiney keymap doesn't compile as a result.

@drashna drashna self-requested a review October 31, 2019 19:57
@noroadsleft
Copy link
Member

Thanks!

@noroadsleft noroadsleft merged commit 77861fc into qmk:master Oct 31, 2019
ripxorip pushed a commit to ripxorip/qmk_firmware that referenced this pull request Dec 3, 2019
* add initial hbcp files

* adjust hbcp keymaps

* update per PR requested changes

* add info.json

* update LED stuff in regards to my personal keymap

* update to new LED pin manipulation code

* update info.json

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/readme.md

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: Joel Challis <git@zvecr.com>

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: Joel Challis <git@zvecr.com>

* update per pull request suggestions

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: Joel Challis <git@zvecr.com>

* update per PR request, update default keymap

* Update keyboards/hineybush/hbcp/keymaps/hiney/keymap.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/keymaps/hiney/keymap.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/keymaps/hiney/keymap.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/info.json

Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>

* Update keyboards/hineybush/hbcp/info.json

Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>

* Update keyboards/hineybush/hbcp/keymaps/hiney/readme.md

Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>

* update per PR requests

* Update keyboards/hineybush/hbcp/info.json

Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>

* add RGB defs to hiney/config.h
HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Feb 21, 2020
* add initial hbcp files

* adjust hbcp keymaps

* update per PR requested changes

* add info.json

* update LED stuff in regards to my personal keymap

* update to new LED pin manipulation code

* update info.json

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/readme.md

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/rules.mk

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: Joel Challis <git@zvecr.com>

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: Joel Challis <git@zvecr.com>

* update per pull request suggestions

* Update keyboards/hineybush/hbcp/hbcp.c

Co-Authored-By: Joel Challis <git@zvecr.com>

* update per PR request, update default keymap

* Update keyboards/hineybush/hbcp/keymaps/hiney/keymap.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/keymaps/hiney/keymap.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/keymaps/hiney/keymap.c

Co-Authored-By: fauxpark <fauxpark@gmail.com>

* Update keyboards/hineybush/hbcp/info.json

Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>

* Update keyboards/hineybush/hbcp/info.json

Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>

* Update keyboards/hineybush/hbcp/keymaps/hiney/readme.md

Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>

* update per PR requests

* Update keyboards/hineybush/hbcp/info.json

Co-Authored-By: noroadsleft <18669334+noroadsleft@users.noreply.github.com>

* add RGB defs to hiney/config.h
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.

6 participants