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

Replace DEBOUNCING_DELAY (depricated) with DEBOUNCE #5997

Merged
merged 1 commit into from
Jun 6, 2019

Conversation

drashna
Copy link
Member

@drashna drashna commented May 27, 2019

Description

Like the title says.

Specifically, DEBOUNCING_DELAY is no longer used by the core code, but DEBOUNCE is.

This changes the defines everywhere to use DEBOUNCE, so it's consistent everywhere.

Types of Changes

  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

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
Member

@fauxpark fauxpark left a comment

Choose a reason for hiding this comment

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

Another day, another ~breaking change~

@noroadsleft
Copy link
Member

Replace DEBOUNCING_DELAY (depricated) with DEBOUNCE

s/depricated/deprecated 😆

@drashna
Copy link
Member Author

drashna commented May 28, 2019

@fauxpark actually, the breaking change has already occurred. DEBOUNCING_DELAY doesn't do anything and defaults to 5ms. This just updates things to be correct.

But yes.

@noroadsleft tomato, tomato. :D

@fauxpark
Copy link
Member

I'm using @skullydazed's definition of "breaking change", because this PR touches user keymaps.

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.

Looks good to me. 👍

@drashna drashna force-pushed the fix/debouncing_defines branch from 1e848d6 to 8f5d8d1 Compare June 5, 2019 18:13
@drashna drashna force-pushed the fix/debouncing_defines branch from 8f5d8d1 to 9d8277d Compare June 5, 2019 18:16
@drashna
Copy link
Member Author

drashna commented Jun 5, 2019

Travis is running make all:all D:

So there will be errors

@mechmerlin mechmerlin merged commit faaaa13 into qmk:master Jun 6, 2019
@drashna drashna deleted the fix/debouncing_defines branch June 6, 2019 19:15
Timbus pushed a commit to Timbus/qmk_firmware that referenced this pull request Jun 23, 2019
doughsay pushed a commit to doughsay/qmk_firmware that referenced this pull request Aug 31, 2019
ridingqwerty pushed a commit to ridingqwerty/qmk_firmware that referenced this pull request Jan 10, 2020
JeffreyPalmer pushed a commit to JeffreyPalmer/qmk_firmware that referenced this pull request Feb 27, 2020
swamp09 pushed a commit to swamp09/qmk_firmware that referenced this pull request Mar 11, 2020
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
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.

5 participants