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

NK Plus #20392

Merged
merged 10 commits into from
Apr 21, 2023
Merged

NK Plus #20392

merged 10 commits into from
Apr 21, 2023

Conversation

yiancar
Copy link
Contributor

@yiancar yiancar commented Apr 9, 2023

Description

Add the NK Plus:)

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

@github-actions github-actions bot added keyboard keymap via Adds via keymap and/or updates keyboard for via support labels Apr 9, 2023
keyboards/novelkeys/nk_plus/config.h Outdated Show resolved Hide resolved
keyboards/novelkeys/nk_plus/config.h Outdated Show resolved Hide resolved
keyboards/novelkeys/nk_plus/keymaps/default/readme.md Outdated Show resolved Hide resolved
keyboards/novelkeys/nk_plus/keymaps/via/keymap.c Outdated Show resolved Hide resolved
keyboards/novelkeys/nk_plus/nk_plus.c Outdated Show resolved Hide resolved
keyboards/novelkeys/nk_plus/nk_plus.h Outdated Show resolved Hide resolved
keyboards/novelkeys/nk_plus/rules.mk Outdated Show resolved Hide resolved
keyboards/novelkeys/nk_plus/rules.mk Outdated Show resolved Hide resolved
@yiancar
Copy link
Contributor Author

yiancar commented Apr 10, 2023

em how did the ci fail on the other ones? any help?

Copy link
Contributor

@elpekenin elpekenin Apr 10, 2023

Choose a reason for hiding this comment

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

imo it makes more sense moving this to keyboards/novelkeys/nk_plus/keymaps/via/readme.md

Copy link
Member

Choose a reason for hiding this comment

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

Also, this is probably why it triggered the odd CI checks

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Deleted to check!

@elpekenin
Copy link
Contributor

em how did the ci fail on the other ones?

No idea about how the CI code works, but since you havent edited those, my first bets would be

  • Some bug
  • They were broken already (i doubt this) and just kept failing

@drashna drashna requested a review from a team April 12, 2023 16:26
@keyboard-magpie keyboard-magpie merged commit ab4013a into qmk:master Apr 21, 2023
@yiancar yiancar mentioned this pull request Apr 21, 2023
7 tasks
@yiancar yiancar deleted the nk_plus branch April 22, 2023 20:26
stratosgear pushed a commit to stratosgear/qmk_firmware that referenced this pull request Apr 26, 2023
struckmb pushed a commit to struckmb/qmk_firmware that referenced this pull request May 1, 2023
rodrigob pushed a commit to rodrigob/qmk_firmware that referenced this pull request May 2, 2023
barrettclark added a commit to barrettclark/qmk_firmware that referenced this pull request May 4, 2023
* upstream/master: (46 commits)
  [Keyboard] Add 1upkeyboards pi50 (qmk#20242)
  `sofle/via` encoder map & doc handedness update (qmk#20630)
  [Keyboard] add phantagom/baragon marcopad (qmk#20543)
  [Keyboard] add phantagom/varan rgb numpad (qmk#20587)
  [Keymap] Small changes to fix failing karn keyboard build (qmk#20618)
  Repair Spleeb i2c config and chrishoage keymap (qmk#20644)
  Adding my keymap (qmk#20345)
  Add additional options for Drag Scroll config (qmk#20523)
  Added Velvet Hotswap and Solder PCBs (qmk#20510)
  [Keyboard] Add the Aster Ergo PCB (qmk#20513)
  vertex/angle65 - Implement RGBlight (qmk#20514)
  [Keymap update] Keymap & docs update (qmk#20298)
  Fix MATRIX_HAS_GHOST when MATRIX_COL > 16 (qmk#20093)
  ez_maker/directpins/promicro: Fix bootloader mismatch (qmk#20627)
  Fix up positioning of keys for Keychron Q10/V10 and Q6/V6 (qmk#20582)
  [Keymap] Add johnstegeman iris colemak keymap (qmk#20575)
  [Keyboard] Add Manibus VIA support (qmk#20497)
  Add KBDCraft Adam64 Lego board (qmk#20503)
  Update code example to match description (qmk#20528)
  NK Plus (qmk#20392)
  ...
SjB added a commit to SjB/qmk_firmware that referenced this pull request May 12, 2023
* master:
  NK Plus (qmk#20392)
  [Docs] Fix suggested code pattern when a specific mod-mask is required. (qmk#20512)
  [Docs] Remove combo count from array (qmk#20511)
  Add QuadrumLabs Delta (qmk#20409)
  Adds Docs option for ArduinoIDE's example `ArduinoISP` (qmk#20486)
  GMMK 2 volume up/down Fn keys are backwards in default mapping (qmk#20476)
  Fix typo in `feature_wpm.md` title (qmk#20464)
  [Keyboard] Add Kalakos Bahrnob65 (qmk#20424)
  Tidy up stray RGB_DISABLE_TIMEOUT references (qmk#20460)
  [Keyboard] Add zoom75 wired (qmk#20396)
  [Keyboard] Add dymium65 (qmk#20257)
  Lotus58 glow (qmk#20386)
  ADPenrose Obi Layout Macro Conversion and Addition (qmk#20445)
autoferrit pushed a commit to SpaceRockMedia/bastardkb-qmk that referenced this pull request Dec 8, 2023
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.

5 participants