Skip to content

[DRAFT] Feat: add re-gret#53

Merged
Nuclear-Squid merged 1 commit into
OneDeadKey:mainfrom
swy7ch:feat/re-gret
Jun 14, 2026
Merged

[DRAFT] Feat: add re-gret#53
Nuclear-Squid merged 1 commit into
OneDeadKey:mainfrom
swy7ch:feat/re-gret

Conversation

@swy7ch

@swy7ch swy7ch commented May 12, 2026

Copy link
Copy Markdown
Contributor

Add support for re-gret (see https://github.com/rschenk/re-gret/tree/main) using RP2040.

@swy7ch

swy7ch commented May 13, 2026

Copy link
Copy Markdown
Contributor Author

Well, it's not going great.

I modified the usual stuff (build.yaml + adding a dedicated keymap), but I'm not sure of what I should do with config/west.yaml. In the original doc for the re-gret, the dedicated module targets zmk@v0.4, which is not the case here I believe (since it's pinned to a particular commit)… Anyhow, it is not clear, and the error message does not help much:

Error: The selected board is not set up for ZMK and there is a
ZMK variant available.
See https://zmk.dev/blog/2025/12/09/zephyr-4-1#zmk-board-variant.

The build can be found here

@swy7ch

swy7ch commented May 13, 2026

Copy link
Copy Markdown
Contributor Author

Well, by un-pinning zmk (which I should not do according to this issue), the build passes (for both the re-gret and the others keebs) as you can see here.

Not sure of what I (we?) should do with that 🤷

@swy7ch

swy7ch commented May 13, 2026

Copy link
Copy Markdown
Contributor Author

There is an issue with ZMK regarding the builds for daughterboards based on RP2040. See this commit and the issue for ækynox here.

We will probably need to bump ZMK before merging this PR

@swy7ch swy7ch force-pushed the feat/re-gret branch 2 times, most recently from 1080813 to f18a1ce Compare May 17, 2026 12:13
@Nuclear-Squid

Copy link
Copy Markdown
Collaborator

Hey, sorry for making you wait. The issue seemed to be a lot harder than what it actually was, so I focused on other tasks first. I’d say, in order to keep the commit history a bit clean, I’ll open up another PR to update ZMK’s version in order to fix your original issue, then we’ll be able to merge your PR directly.

@swy7ch

swy7ch commented Jun 13, 2026

Copy link
Copy Markdown
Contributor Author

Hey! No worries, I have ip working on my end so there’s no need to rush. I opened the issue #55 in that regard, to know how we want to do it. I’d gladly help :)

@swy7ch swy7ch force-pushed the feat/re-gret branch 6 times, most recently from 8d8060a to 6bd808c Compare June 14, 2026 08:35
@swy7ch

swy7ch commented Jun 14, 2026

Copy link
Copy Markdown
Contributor Author

I rebased against main and just flashed the keeb, but the alt-gr layer seems bugged. For instance, alt+s gives #, but should give a (. Here’s the output of the alt layer:

^G;$% «&*'`
{#@}= \+/k"
~[]_» |€uUK

I wonder if the problem is the macro FOUR_THUMB_KEYS, defined in config/re-gret.keymap. It is adapted from config/ferris.keymap. see here, but the macro does not seem to be used in the repo. Indeed, aekynox/settings.h defines HT_TWO_THUMB_KEYS.

Using the HT_TWO_THUMB_KEYS does not fix the issue. Maybe setting this in the keymap does carry on to the final build? Setting it in settings.h behaves as expected.

I must point out that I have no quacken or other supported keeb to check if it is a global issue or specific to the re-gret.

It’s all good, I simply forgot to uncomment #define KB_LAYOUT_ERGOL in aekynox/settings.h 🙃

@swy7ch swy7ch force-pushed the feat/re-gret branch 2 times, most recently from a340563 to 0f47bf0 Compare June 14, 2026 12:31
@swy7ch

swy7ch commented Jun 14, 2026

Copy link
Copy Markdown
Contributor Author

The build passed (see https://github.com/swy7ch/zmk-config-aekeynox/actions/runs/27498925972), everything is fine as far as I’m concerned. Marking the PR ready!

@swy7ch swy7ch marked this pull request as ready for review June 14, 2026 12:35

@Nuclear-Squid Nuclear-Squid left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, sorry again for making you wait

@Nuclear-Squid Nuclear-Squid merged commit 8c4ff62 into OneDeadKey:main Jun 14, 2026
72 checks passed
niiico69 pushed a commit to niiico69/zmk-config-aekeynox that referenced this pull request Jun 17, 2026
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