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

[keymap(kyria), refactor] created kyria keymap. refactored ninjonas userspace #8053

Merged
merged 5 commits into from
Feb 1, 2020
Merged

[keymap(kyria), refactor] created kyria keymap. refactored ninjonas userspace #8053

merged 5 commits into from
Feb 1, 2020

Conversation

ninjonas
Copy link
Contributor

Description

  • Adding new kyria keymap
  • Minor tweaks to ninjonas userspace

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

Copy link
Contributor

@Duckle29 Duckle29 left a comment

Choose a reason for hiding this comment

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

EDIT: Brainfart. didn't mean to approve. just that small note below

keyboards/kyria/keymaps/ninjonas/rules.mk Outdated Show resolved Hide resolved
Copy link
Contributor

@Duckle29 Duckle29 left a comment

Choose a reason for hiding this comment

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

The change requested above ^

@ninjonas
Copy link
Contributor Author

The change requested above ^

Thanks @Duckle29 for the code review. Just pushed the changes requested.

@Duckle29 Duckle29 self-requested a review January 31, 2020 21:48
@Duckle29 Duckle29 requested a review from a team January 31, 2020 21:48
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.

A couple of your keycode assignments looked incorrect to me, but in general, looks good.

users/ninjonas/ninjonas.h Outdated Show resolved Hide resolved
users/ninjonas/ninjonas.h Outdated Show resolved Hide resolved
@noroadsleft noroadsleft merged commit 1aee492 into qmk:master Feb 1, 2020
@noroadsleft
Copy link
Member

Thanks!

HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Feb 21, 2020
…serspace (qmk#8053)

* [keymap(kyria), refactor] created kyria keymap. refactored ninjonas userspace and keyboards

* [fix(qmk#8053)] removed redundant code based off Duckle29's code review

* [fix(qmk#8053)] fixing typos

* [refactor(qmk#8053)] updating oled timeout

* [fix(8053) resolved code review issues identified by noroadsleft
c0psrul3 pushed a commit to c0psrul3/qmk_firmware that referenced this pull request Mar 23, 2020
…serspace (qmk#8053)

* [keymap(kyria), refactor] created kyria keymap. refactored ninjonas userspace and keyboards

* [fix(qmk#8053)] removed redundant code based off Duckle29's code review

* [fix(qmk#8053)] fixing typos

* [refactor(qmk#8053)] updating oled timeout

* [fix(8053) resolved code review issues identified by noroadsleft
kylekuj pushed a commit to kylekuj/qmk_firmware that referenced this pull request Apr 21, 2020
…serspace (qmk#8053)

* [keymap(kyria), refactor] created kyria keymap. refactored ninjonas userspace and keyboards

* [fix(qmk#8053)] removed redundant code based off Duckle29's code review

* [fix(qmk#8053)] fixing typos

* [refactor(qmk#8053)] updating oled timeout

* [fix(8053) resolved code review issues identified by noroadsleft
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
…serspace (qmk#8053)

* [keymap(kyria), refactor] created kyria keymap. refactored ninjonas userspace and keyboards

* [fix(qmk#8053)] removed redundant code based off Duckle29's code review

* [fix(qmk#8053)] fixing typos

* [refactor(qmk#8053)] updating oled timeout

* [fix(8053) resolved code review issues identified by noroadsleft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants