Skip to content

fix(docs): variants file names and directory structure#159

Merged
DhruvaG2000 merged 1 commit intozephyrproject-rtos:nextfrom
soburi:doc_update
Feb 7, 2026
Merged

fix(docs): variants file names and directory structure#159
DhruvaG2000 merged 1 commit intozephyrproject-rtos:nextfrom
soburi:doc_update

Conversation

@soburi
Copy link
Copy Markdown
Member

@soburi soburi commented Feb 6, 2026

No description provided.

Co-authored-by: Gilberto Conti <g.conti@arduino.cc>
Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
Copilot AI review requested due to automatic review settings February 6, 2026 12:50
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes inaccurate documentation in the variants guide by updating the instructions and directory structure to reflect the actual implementation. The changes remove references to files that don't exist (pinmap header files, CMakeLists.txt, and variant.h at the variants root level) and simplify the documentation to show only the two files that are actually required: an overlay file and a variant.h file within each board-specific directory.

Changes:

  • Simplified instructions for adding board variants, removing reference to non-existent pinmap header files
  • Updated directory structure example to show only the overlay and variant.h files that actually exist
  • Removed detailed but inaccurate explanations about CMakeLists.txt and variant.h at the variants root level

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@soburi soburi requested a review from DhruvaG2000 February 6, 2026 21:54
@DhruvaG2000 DhruvaG2000 merged commit 5f797d7 into zephyrproject-rtos:next Feb 7, 2026
9 checks passed
@soburi soburi deleted the doc_update branch February 7, 2026 04:49
@soburi soburi mentioned this pull request Feb 14, 2026
16 tasks
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.

4 participants