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
feat(docs): Add New Behavior Guide #1268
Conversation
Co-Authored-By: Henré Botha <henrebotha@gmail.com> Co-Authored-By: Cem Aksoylar <caksoylar@users.noreply.github.com> Co-Authored-By: Dom H <dom@hxy.io>
|
||
LOG_MODULE_DECLARE(zmk, CONFIG_ZMK_LOG_LEVEL); | ||
|
||
#if DT_HAS_COMPAT_STATUS_OKAY(DT_DRV_COMPAT) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Planning to rewrite this section given the review done by @joelspadin in #1275.
#1275 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leave it for a separate MR maybe? I think it would be good to just get this merged and then follow up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome guide – looks great to me from a docs point of view!
As mentioned in #1174 (review), I think it would be good to get approval from at least one developer contributor familiar with the more technical aspects. Maybe @petejohanson, @joelspadin or @okke-formsma could take a glance over this and give a thumbs up before we merge?
- Address Event Manager review comments by @okke-formsma - Update PR section to link to contributor documentation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple minor things. On a whole this is AWESOME.
f9747f2
to
71f6c67
Compare
- Update hardware testing section - Add note to update app/dts/behaviors.dtsi Co-Authored-By: Pete Johanson <peter@peterjohanson.com>
71f6c67
to
a04c599
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one other thing I noticed.
826ef97
to
523ae83
Compare
50cd342
to
02b611e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awesome. @dxmh or @caksoylar can weight in/approve and merge when they're ready.
02b611e
to
052c700
Compare
👷 Deploy request for zmk accepted.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am curious if the removal of the inline code formatting (backticks) was intentional in the latest commit, 052c700. But either way, all the recent changes since my last review look great to me. 👍
2082fe5
to
8518149
Compare
Co-Authored-By: Henré Botha henrebotha@gmail.com
Co-Authored-By: Cem Aksoylar caksoylar@users.noreply.github.com
Co-Authored-By: Dom H dom@hxy.io
This PR adds a guide for developers wishing to create new behaviors for ZMK. The behavior guide is largely inspired by the new shield guide, and shows users how to: