-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
doc: improvements to nRF Desktop and CAF documentation #19719
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:more detailsGithub labels
List of changed files detected by CI (0)
Outputs:ToolchainVersion: Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publish GitHub Action. |
967a8c1
to
e261d34
Compare
a57ea3b
to
e7467bb
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.
Overall LGTM.
I am not sure if not adding "Kconfig" before "option" in following places is intentional, but just in case i marked those places (I think that in most cases there is full "Kconfig option" instead of just "option").
Can be ignored if intentional :D
Making configuration steps clearer. Also other minor fixes. NCSDK-30075 Signed-off-by: Pekka Niskanen <[email protected]>
Making configuration steps clearer.
Also other minor fixes.
NCSDK-30075