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

add minimal workflow for keymap drawer #67

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stijnveenman
Copy link

Thanks again for the awesome config and write up about it, it gave me a great starting point for my configuration. Consider this my thank you note. :)

It is now possible to use keymap drawer with v2 of zmk-helpers

This PR shows a minimal implementation of it. There is still more configuration required to make it look decent.
With some inspiration of https://github.com/minusfive/zmk-config i have a more complete config on my fork of your config.

Since i made quite some changes to your config, also removing some keys, my config is probably still incomplete for yours. But might give a more complete starting point. See: https://github.com/stijnveenman/zmk-config/blob/main/keymap_drawer.config.yaml

Which renders:
image

An example of this minimal workflow can be found at: https://github.com/stijnveenman/zmk-config/actions/runs/9920716711

Note: keymap-drawer renders all .keymap files by default (configured in yaml),so with your multiple keyboard configurations it will try and render base.keymap which will fail. But it will still continue with the rest of them. Since i only have one keyboard i hardcoded mine

@urob
Copy link
Owner

urob commented Jul 19, 2024

Loving this! I am somewhat buried in work these days so it might take a bit before I'll get to incorporating this here. But I'll get there... :)

@urob urob force-pushed the main branch 4 times, most recently from 898f77c to be30faf Compare August 14, 2024 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants