fix: rename agent/master comm doc to agent/server, modernize content and diagram#256
Draft
miharp wants to merge 3 commits into
Draft
fix: rename agent/master comm doc to agent/server, modernize content and diagram#256miharp wants to merge 3 commits into
miharp wants to merge 3 commits into
Conversation
- Rename subsystem_agent_master_comm.markdown to subsystem_agent_server_comm.markdown - Rewrite content to adopt Puppet docs structure: single numbered flow (steps 1-6), updated cert acquisition steps (CRL, CSR with dns_alt_names/csr_attributes, waitforcert behavior), cleaner keep-alive section, accurate intro - Replace outdated GIF diagram with Mermaid sequence diagram; remove old GIF files - Add Mermaid rendering support to head_end.html (CDN, pinned to 11.15.0) - Add package.json and Dependabot npm tracking for Mermaid version management - Update inbound links in architecture.markdown, ssl_attributes_extensions.markdown, and nav file; drop unnecessary ./ prefix per CONTRIBUTING.md link rules Signed-off-by: Michael Harp <mike@mikeharp.com>
File was touched to update link text; per project convention, clean all violations in the changed file: wrap long lines (MD013), add language identifiers to fenced code blocks (MD040), suppress false-positive emphasis warning around Liquid include tag (MD037), and drop unnecessary ./ prefixes from link reference definitions per CONTRIBUTING.md linking rules. Signed-off-by: Michael Harp <mike@mikeharp.com>
miharp
force-pushed
the
fix/agent-server-comm-terminology
branch
from
770c7f7 to
b1de12a
Compare
File was touched to update link URL; per project convention, clean all
violations in the changed file: wrap long lines (MD013), add blank lines
around headings (MD022), fix unused/undefined reference definitions
(MD053/MD052), drop unnecessary ./ prefixes per CONTRIBUTING.md linking
rules, and remove unused Kramdown IAL annotations ({:.concept}/{:.section})
that have no effect in the current theme.
Signed-off-by: Michael Harp <mike@mikeharp.com>
Contributor
|
Yay mermaid! 🧜♀️ @OpenVoxProject/sig-documentation we are going to need to decide whether we're doing sembr and if so, what variant. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
subsystem_agent_master_comm.markdown→subsystem_agent_server_comm.markdownand update all inbound links (nav,architecture.markdown,ssl_attributes_extensions.markdown)dns_alt_names/csr_attributes,waitforcertbehavior), cleaner keep-alive sectionhead_end.html(CDN module import, Turbo + window.load event handling)11.15.0in both the CDN URL and a newpackage.json; add npm ecosystem to Dependabot so version bump PRs are raised automaticallyssl_attributes_extensions.markdown(touched file)Test plan
architecture.markdownandssl_attributes_extensions.markdownresolve correctly in built outputPart of #223