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

LEADER_NO_TIMEOUT_N #24581

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from
Open

LEADER_NO_TIMEOUT_N #24581

wants to merge 14 commits into from

Conversation

Mr-Asmodaeus
Copy link

@Mr-Asmodaeus Mr-Asmodaeus commented Nov 9, 2024

Added a definable LEADER_NO_TIMEOUT_FOR_N_KEYSTROKES to leader_key.c that allows disabling timeout for the initial N keys in the sequence (including leader)

Description

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout (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).

…abling the timeout for the first N keystrokes in leader key sequences.
@Mr-Asmodaeus Mr-Asmodaeus marked this pull request as ready for review November 9, 2024 19:31
@Mr-Asmodaeus
Copy link
Author

Recreated after cleaning up my repo. Sorry for the earlier mess.

docs/features/leader_key.md Outdated Show resolved Hide resolved
Copy link
Member

@drashna drashna left a comment

Choose a reason for hiding this comment

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

Since leader keys already have unit tests, any additional functionality should include additional tests to verify that the new functionality works as expected.

@zvecr zvecr changed the base branch from master to develop November 10, 2024 01:29
@zvecr
Copy link
Member

zvecr commented Nov 10, 2024

From the PR checklist,

all core PRs must now target develop branch, which will subsequently be merged back to master on the breaking changes timeline

I have updated the PR to reflect this.

@Mr-Asmodaeus Mr-Asmodaeus requested a review from drashna November 16, 2024 15:28
@Mr-Asmodaeus
Copy link
Author

Is there anything missing on my end ?

@drashna drashna requested a review from a team December 6, 2024 02:48
docs/features/leader_key.md Show resolved Hide resolved
docs/features/leader_key.md Outdated Show resolved Hide resolved
docs/features/leader_key.md Outdated Show resolved Hide resolved
docs/features/leader_key.md Outdated Show resolved Hide resolved
quantum/leader.c Outdated Show resolved Hide resolved
quantum/leader.c Outdated Show resolved Hide resolved
quantum/leader.c Outdated Show resolved Hide resolved
@Mr-Asmodaeus Mr-Asmodaeus requested a review from fauxpark December 7, 2024 10:39
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.

4 participants