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

Remove unnecessary IS_COMMAND definitions from a couple of boards #5269

Merged
merged 4 commits into from
Mar 19, 2019

Conversation

vomindoraan
Copy link
Contributor

@vomindoraan vomindoraan commented Feb 28, 2019

Another few outdated IS_COMMAND definitions that made it past code review. See #4301 #5065 for more information.

@vomindoraan vomindoraan changed the title Remove unnecessary IS_COMMAND definition from clueboard/66_hotswap/gen1 Remove unnecessary IS_COMMAND definition from clueboard/66_hotswap/gen1 and dozen0 Mar 2, 2019
@vomindoraan vomindoraan changed the title Remove unnecessary IS_COMMAND definition from clueboard/66_hotswap/gen1 and dozen0 Remove unnecessary IS_COMMAND definitions from a couple of boards Mar 2, 2019
@drashna
Copy link
Member

drashna commented Mar 5, 2019

@skullydazed Any issues with this?

Just wanted to ping you, since it's your board.

@vomindoraan
Copy link
Contributor Author

Bump.

@drashna
Copy link
Member

drashna commented Mar 19, 2019

Thanks

@drashna drashna merged commit 6a4884e into qmk:master Mar 19, 2019
loadedsith added a commit to loadedsith/qmk_firmware that referenced this pull request Mar 19, 2019
* master: (476 commits)
  [Keymap] Boy_314's Halfnhalf layout update (qmk#5441)
  [Keyboard] Remove unnecessary IS_COMMAND definitions from a couple of boards (qmk#5269)
  Cleanup Default Keymaps  (qmk#5442)
  Fix dk60 compiler issue on newer compiler versions (qmk#5444)
  correct typo in eeprom split target for dfu (qmk#5445)
  [Docs] Recursively clone qmk_firmware repo (qmk#5378)
  [Docs] Smallish overhaul of the docs (qmk#5281)
  [Keyboard] Reduce compile size of the Lily58 Keyboard (qmk#5412)
  [Docs] Add music_map to audio documentation (qmk#5419)
  [Keymap] PyroL's TMO50 layout update (qmk#5435)
  [Keymap] Add vim-style keymap for lets split (qmk#5399)
  [Keyboard] CannonKeys Instant60 Keyboard (qmk#5433)
  [Keymap] add android studio keyboard shortcuts to dvorak 42 key atreus / ergodox layouts (qmk#5423)
  [Keymap] Updated layout after some months of testing. (qmk#5427)
  [Keyboard] Corrected layout for top row keys (qmk#5429)
  [Keyboard] Add Jot50 handwired board (qmk#5431)
  [Keyboard] layout fixed in jotanck.h (qmk#5432)
  [Keyboard] correct indicator light states (qmk#5434)
  [Keyboard] Fix matrix positions for HotDox (qmk#5437)
  [Keymap] Arkag Userspace Update (qmk#5438)
  ...

# Conflicts:
#	keyboards/keebio/iris/rules.mk
#	layouts/community/ortho_4x12/grahampheath/keymap.c
chie4hao pushed a commit to chie4hao/qmk_firmware that referenced this pull request Mar 28, 2019
… boards (qmk#5269)

* Remove unnecessary IS_COMMAND definition from clueboard/66_hotswap/gen1

* Remove old-style IS_COMMAND definition comment from dozen0

* Use get_mods() instead of keyboard_report->mods in georgi
zer09 pushed a commit to zer09/qmk_firmware that referenced this pull request Mar 31, 2019
… boards (qmk#5269)

* Remove unnecessary IS_COMMAND definition from clueboard/66_hotswap/gen1

* Remove old-style IS_COMMAND definition comment from dozen0

* Use get_mods() instead of keyboard_report->mods in georgi
danielo515 pushed a commit to danielo515/qmk_firmware that referenced this pull request May 15, 2019
… boards (qmk#5269)

* Remove unnecessary IS_COMMAND definition from clueboard/66_hotswap/gen1

* Remove old-style IS_COMMAND definition comment from dozen0

* Use get_mods() instead of keyboard_report->mods in georgi
Timbus pushed a commit to Timbus/qmk_firmware that referenced this pull request Jun 23, 2019
… boards (qmk#5269)

* Remove unnecessary IS_COMMAND definition from clueboard/66_hotswap/gen1

* Remove old-style IS_COMMAND definition comment from dozen0

* Use get_mods() instead of keyboard_report->mods in georgi
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.

2 participants