Skip to content

wolfHSM updates#266

Merged
padelsbach merged 1 commit into
wolfSSL:masterfrom
bigbrett:wolfhsm-docs-v2
May 29, 2026
Merged

wolfHSM updates#266
padelsbach merged 1 commit into
wolfSSL:masterfrom
bigbrett:wolfhsm-docs-v2

Conversation

@bigbrett
Copy link
Copy Markdown
Contributor

@bigbrett bigbrett commented May 29, 2026

updates to work with the new wolfHSM doc structure (english only)

Required to properly render wolfSSL/wolfHSM#387 (CI will fail until that PR is merged)

@bigbrett bigbrett self-assigned this May 29, 2026
Copilot AI review requested due to automatic review settings May 29, 2026 15:22
@bigbrett bigbrett marked this pull request as draft May 29, 2026 15:23
Copy link
Copy Markdown
Contributor

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

Updates wolfHSM documentation build configuration for the new upstream English documentation structure while keeping the Japanese source layout conditionalized separately.

Changes:

  • Replaces English chapter lists and mkdocs navigation with new numbered source files.
  • Adds new client/server API reference pages and Doxygen inputs.
  • Updates Makefile preprocessing for new links and split API output.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
wolfHSM/mkdocs.yml Updates English site navigation for the new documentation structure and expanded API pages.
wolfHSM/Makefile Adds language-specific source lists and preprocessing for new English docs/API output.
wolfHSM/Doxyfile Adds additional wolfHSM headers to Doxygen input.

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

Comment thread wolfHSM/Makefile Outdated
@bigbrett bigbrett marked this pull request as ready for review May 29, 2026 17:40
@padelsbach padelsbach merged commit b5f8c59 into wolfSSL:master May 29, 2026
2 checks passed
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