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

Bandominedoni version output feature and a new keymap added #17947

Merged
merged 7 commits into from
Aug 13, 2022

Conversation

3araht
Copy link
Contributor

@3araht 3araht commented Aug 8, 2022

Description

A feature to output the name of the keyboard, keymap, QMK version, and the build date is added.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • [x ] 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 keymap via Adds via keymap and/or updates keyboard for via support labels Aug 8, 2022
@@ -28,6 +29,10 @@ enum layer_names {
_FN
};

enum custom_keycodes {
VERSION = SAFE_RANGE
Copy link
Member

Choose a reason for hiding this comment

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

This would allow you to list it in VIA

Suggested change
VERSION = SAFE_RANGE
VERSION = USER00

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wow! That’s cool. We’ve been waiting for such a feature on VIA!
I’ll update my codes as you suggested.

It seems you can use this as long as you have 16 or less custom codes.
https://github.com/qmk/qmk_firmware/blob/master/quantum/via.h

@drashna drashna requested a review from a team August 8, 2022 23:26
@keyboard-magpie keyboard-magpie merged commit e145c7a into qmk:master Aug 13, 2022
@3araht 3araht deleted the bandominedoni_version_added branch August 14, 2022 00:10
imhoffman pushed a commit to imhoffman/qmk_firmware that referenced this pull request Aug 20, 2022
* SPLIT_USB_DETECT added.

* lednotg keymap added.

* lednotg missing modification fixed.

* VERSION is available.

* USER00 is used instead of SAFE_RANGE in via/keymap.c
nolanseaton pushed a commit to nolanseaton/qmk_firmware that referenced this pull request Jan 23, 2023
* SPLIT_USB_DETECT added.

* lednotg keymap added.

* lednotg missing modification fixed.

* VERSION is available.

* USER00 is used instead of SAFE_RANGE in via/keymap.c
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.

3 participants