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

Split hand pin change for splinky v3 #31

Merged
merged 9 commits into from
Nov 9, 2022

Conversation

AlexBaldwin42
Copy link

Moved the MASTER_RIGHT and the commented-out code for the SPLIT_HAND_PIN and SPLIT_HAND_PIN_LOW_IS_LEFT to the respective v2 v3 config.h files as the pin was changed from v2 to v3.

Copy link
Collaborator

@0xcharly 0xcharly left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

Would you mind also doing this for the Scylla, TBK mini and Skeletyl? It would be great to have all these in a single PR.

keyboards/bastardkb/charybdis/3x5/v2/splinky/v2/config.h Outdated Show resolved Hide resolved
keyboards/bastardkb/charybdis/3x5/v2/splinky/v3/config.h Outdated Show resolved Hide resolved
keyboards/bastardkb/charybdis/3x6/v2/splinky/v2/config.h Outdated Show resolved Hide resolved
keyboards/bastardkb/charybdis/3x6/v2/splinky/v3/config.h Outdated Show resolved Hide resolved
keyboards/bastardkb/charybdis/4x6/v2/splinky/v2/config.h Outdated Show resolved Hide resolved
keyboards/bastardkb/charybdis/4x6/v2/splinky/v3/config.h Outdated Show resolved Hide resolved
AlexBaldwin42 and others added 6 commits November 7, 2022 22:34
Co-authored-by: Charly Delay <0xcharly@users.noreply.github.com>
Co-authored-by: Charly Delay <0xcharly@users.noreply.github.com>
Co-authored-by: Charly Delay <0xcharly@users.noreply.github.com>
Co-authored-by: Charly Delay <0xcharly@users.noreply.github.com>
Co-authored-by: Charly Delay <0xcharly@users.noreply.github.com>
Co-authored-by: Charly Delay <0xcharly@users.noreply.github.com>
@AlexBaldwin42
Copy link
Author

No problem! I will do those as well.

@0xcharly 0xcharly merged this pull request into Bastardkb:bkb-master Nov 9, 2022
0xcharly added a commit that referenced this pull request Nov 9, 2022
Fix split-hand pin hint for boards supporting v2 an v3 of the Splinky

Co-authored-by: Simian <alex@Mandelbrot>
Co-authored-by: Charly Delay <0xcharly@users.noreply.github.com>
0xcharly added a commit that referenced this pull request Nov 9, 2022
Fix split-hand pin hint for boards supporting v2 an v3 of the Splinky

Co-authored-by: Simian <alex@Mandelbrot>
Co-authored-by: Charly Delay <0xcharly@users.noreply.github.com>
0xcharly added a commit that referenced this pull request Nov 14, 2022
Fix split-hand pin hint for boards supporting v2 an v3 of the Splinky

Co-authored-by: Simian <alex@Mandelbrot>
Co-authored-by: Charly Delay <0xcharly@users.noreply.github.com>
0xcharly added a commit that referenced this pull request Nov 17, 2022
Fix split-hand pin hint for boards supporting v2 an v3 of the Splinky

Co-authored-by: Simian <alex@Mandelbrot>
Co-authored-by: Charly Delay <0xcharly@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants