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

KP Republic BM60HSRGB_EC QMK Configurator and readme touch-up #17541

Merged
merged 4 commits into from
Jul 3, 2022

Conversation

noroadsleft
Copy link
Member

Description / Issues Fixed or Closed by This PR

QMK Configurator updates

rev1 rev2
apply friendly formatting ✔️
adjust layout data to convey encoder rotation actions ✔️ ✔️
specify keyboard revision in keyboard_name ✔️ ✔️
update maintainer value ✔️ ✔️
remove meta key ✔️

Before (rev1 shown):
image
After:
image

Readme updates

rev1 rev2
correct Hardware Availability links (previously linked to wrong product page) ✔️ ✔️
Markdown formatting update ✔️

Change listed maintainer

Previously listed maintainer hasn't ever committed to this codebase... so I changed it.

cc @peepeetee (new keyboard maintainer)

Types of Changes

  • Enhancement/optimization
  • Keyboard (addition or update)

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.
  • I have read the CONTRIBUTING document.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

- apply friendly formatting
- adjust layout data to convey encoder rotation actions in QMK Configurator
- specify rev1 in `keyboard_name`
- update `maintainer` value
- remove `meta` key
- adjust layout data to convey encoder rotation actions in QMK Configurator
- specify rev2 in `keyboard_name`
- update `maintainer` value
- correct Hardware Availability links (previously linked to wrong product page)
- Markdown formatting update for `rev2/readme.md`
Previously listed maintainer has never committed to this codebase.
@drashna drashna requested a review from a team July 2, 2022 19:06
@drashna drashna merged commit 9a157a3 into qmk:master Jul 3, 2022
@noroadsleft noroadsleft deleted the cf/kprepublic_bm60hsrgb_ec branch July 4, 2022 17:51
schattenbrot pushed a commit to schattenbrot/qmk_firmware that referenced this pull request Aug 2, 2022
nolanseaton pushed a commit to nolanseaton/qmk_firmware that referenced this pull request Jan 23, 2023
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.

3 participants