-
-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
Cleanup check for PERMISSIVE_HOLD #7861
Conversation
caused by #5009 (aka, me)
How many people outside the repo have defined PER_KEY_TAPPING_TERM? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Chances are very low that anyone defined PER_KEY_TAPPING_TERM
since this is the only existence of that phrasing, so it’s unlikely to unexpectedly alter behavior for anyone.
Exactly. It's ... an artifact of me try getting the feature to work, where I changed the name. It's not used anywhere else. |
Its not used anywhere else in That aside, I do agree that the impact is very low, but i cant see it as zero. |
* Fix bug in PERMISSIVE_HOLD check caused by qmk#5009 (aka, me) * Remove check for per key
* Fix bug in PERMISSIVE_HOLD check caused by qmk#5009 (aka, me) * Remove check for per key
* Fix bug in PERMISSIVE_HOLD check caused by qmk#5009 (aka, me) * Remove check for per key
* Fix bug in PERMISSIVE_HOLD check caused by qmk#5009 (aka, me) * Remove check for per key
* Fix bug in PERMISSIVE_HOLD check caused by qmk#5009 (aka, me) * Remove check for per key
* Fix bug in PERMISSIVE_HOLD check caused by qmk#5009 (aka, me) * Remove check for per key
Description
Extra, unneeded check
Types of Changes
Issues Fixed or Closed by This PR