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

Fix missing wait.h include in Dip Switch Map #24863

Merged
merged 1 commit into from
Jan 25, 2025

Conversation

drashna
Copy link
Member

@drashna drashna commented Jan 25, 2025

Description

While playing with test board, ran into "implicit definition of wait_ms" when dip switch map is enabled and TAP_CODE_ENABLE or DIP_SWITCH_MAP_KEY_DELAY is configured.

Considering it hasn't come up before now....

Types of Changes

  • Core
  • Bugfix

Issues Fixed or Closed by This PR

  • compilation issue

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

@drashna drashna requested a review from a team January 25, 2025 00:38
Copy link
Member

@zvecr zvecr left a comment

Choose a reason for hiding this comment

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

We could push this directly to master as its a fairly trivial bug fix?

@drashna
Copy link
Member Author

drashna commented Jan 25, 2025

We could push this directly to master as its a fairly trivial bug fix?

Can do. But considering it's been in there a while and doesn't seem to have come up before now, I wasn't "in a hurry".

@drashna drashna changed the base branch from develop to master January 25, 2025 01:03
@drashna drashna force-pushed the fix/dip_switch_map branch from 0d8d720 to b81ecc3 Compare January 25, 2025 01:03
@zvecr zvecr merged commit 6a9ccae into qmk:master Jan 25, 2025
4 checks passed
@drashna drashna deleted the fix/dip_switch_map branch January 25, 2025 03:44
alk-alyss added a commit to alk-alyss/qmk_firmware that referenced this pull request Jan 30, 2025
* 'master' of https://github.com/qmk/qmk_firmware: (221 commits)
  Add Keychron C3 Pro (Red & RGB) (qmk#24874)
  [OS Detection] Improve MacOS detection (qmk#24708)
  Bump vite from 5.2.14 to 5.4.12 in /builddefs/docsgen (qmk#24852)
  Reject keyboard.json next to info.json (qmk#24882)
  Add keyboard clap_studio/flame60 (qmk#24870)
  [Keyboard] Add Support E8ghtyNeo (qmk#24859)
  [License Violation] add pressplayid (qmk#24869)
  Fix missing wait.h include in Dip Switch Map (qmk#24863)
  Add Model-B keyboard (qmk#24681)
  Updated Licence Violations for Chosfox L75 (qmk#24833)
  License violations updates. (qmk#24831)
  Add Lily (qmk#24813)
  `kikoslab/kl90`: fix keymap (qmk#24826)
  [Keyboard] Add ic45v2 (qmk#24818)
  Allow running Docker container on MacOS (without in-container USB support) (qmk#24412)
  `salane/ncr80alpsskfl`: add additional layouts (qmk#24809)
  Bump nanoid from 3.3.7 to 3.3.8 in /builddefs/docsgen (qmk#24713)
  Fix a build warning and tidy up a keymap (qmk#24805)
  [Bug] Fix Underglow keycode processing (qmk#24798)
  [Docs] Fix wording in Mod Tap documentation (qmk#24796)
  ...
Rumadon pushed a commit to Rumadon/qmk_firmware that referenced this pull request Jan 31, 2025
MaximumP pushed a commit to MaximumP/qmk_firmware that referenced this pull request Feb 1, 2025
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