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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds keymaps for muzfuz DZ60, Planck, Clueboard66 #4825

Merged
merged 3 commits into from Jan 13, 2019
Merged

Conversation

muzfuz
Copy link
Contributor

@muzfuz muzfuz commented Jan 11, 2019

Adds keymaps for Planck, DZ60 and Clueboard66
馃憤

@muzfuz muzfuz changed the title Adds keymaps for muzfuz Adds keymaps for muzfuz DZ60, Planck, Clueboard66 Jan 11, 2019
keyboards/clueboard/66/keymaps/muzfuz/keymap.c Outdated Show resolved Hide resolved
keyboards/clueboard/66/keymaps/muzfuz/keymap.c Outdated Show resolved Hide resolved
keyboards/clueboard/66/keymaps/muzfuz/keymap.c Outdated Show resolved Hide resolved
keyboards/dz60/keymaps/muzfuz/keymap.c Outdated Show resolved Hide resolved
keyboards/clueboard/66/keymaps/muzfuz/keymap.c Outdated Show resolved Hide resolved
keyboards/clueboard/66/keymaps/muzfuz/rules.mk Outdated Show resolved Hide resolved
keyboards/dz60/keymaps/muzfuz/keymap.c Outdated Show resolved Hide resolved

extern keymap_config_t keymap_config;

enum planck_layers
Copy link
Member

Choose a reason for hiding this comment

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

This is optional, and mor of a nitpick (so you can ignore).

Would you be willing to reformat this so it's conforming to the QMK Coding conventions?

https://docs.qmk.fm/#/contributing?id=coding-conventions

keyboards/planck/keymaps/muzfuz/keymap.c Outdated Show resolved Hide resolved
@muzfuz
Copy link
Contributor Author

muzfuz commented Jan 13, 2019

@drashna @mechmerlin thank you for the reviews. 馃槏 I have made appropriate changes.

The only one I didn't change is the formatting request in the Planck keymap, as this is in line with the default template at the moment.

@drashna
Copy link
Member

drashna commented Jan 13, 2019

Thanks!

@drashna drashna merged commit a9982e3 into qmk:master Jan 13, 2019
loadedsith added a commit to loadedsith/qmk_firmware that referenced this pull request Jan 15, 2019
* master: (759 commits)
  [Keyboard] Add cKeys Handwire 101 Keyboard (qmk#4848)
  [Keyboard] Iris via support, Rev 3 updates (qmk#4849)
  [Keyboard] Add bthlabs/geekpad (qmk#4840)
  [Keymap] Added resfury keymap (qmk#4827)
  Pointed LM Docs at expected keycodes (qmk#4835)
  Add personal userspace, update keymaps (qmk#4845)
  [Keyboard] Add support for THE50 (qmk#4844)
  Change handling of CUSTOM_MATRIX in common_features.mk slightly.
  Modified URLs to point to new locations
  [Keymap] Nyquist layout adapted from eorgodox_ez:skug (qmk#4830)
  [Keymap] Adds keymaps for muzfuz DZ60, Planck, Clueboard66 (qmk#4825)
  Changed rest note (qmk#4837)
  [Keymap] Add tw1t611 german keyboard layout for minidox. (qmk#4679)
  [Keyboard] hexon38 and Dual-role key implementation (qmk#4709)
  [Keymap] my keymap for the crkbd and update my iris keymap readme (qmk#4788)
  [Keymap] Update to personal keymaps and userspace (qmk#4831)
  [Keyboard] Fix layout macro name for Gergo info.json (qmk#4828)
  Remove empty action_function()
  Remove empty fn_actions[]
  [Keyboard] Adding support for Gergo (qmk#4792)
  ...
rseymour pushed a commit to rseymour/qmk_firmware that referenced this pull request Mar 13, 2019
* Adds keymaps for muzfuz

* Remove unused keys from adjust layer

* Bring in line with current QMK standards.
dlhextall pushed a commit to dlhextall/qmk_firmware that referenced this pull request May 24, 2019
* Adds keymaps for muzfuz

* Remove unused keys from adjust layer

* Bring in line with current QMK standards.
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.

None yet

3 participants