Skip to content

feat(pass): documentation#514

Merged
joe0BAB merged 1 commit intomainfrom
feat/docs
Apr 23, 2026
Merged

feat(pass): documentation#514
joe0BAB merged 1 commit intomainfrom
feat/docs

Conversation

@joe0BAB
Copy link
Copy Markdown
Collaborator

@joe0BAB joe0BAB commented Apr 23, 2026

No description provided.

Signed-off-by: Johannes Großmann <grossmann.johannes@t-online.de>
@joe0BAB joe0BAB marked this pull request as ready for review April 23, 2026 14:05
Copy link
Copy Markdown

@docker-agent docker-agent Bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This PR replaces # shell-comment markers with ### Markdown heading markers in Cobra Example strings. No logic, security, or functional issues are introduced. Two minor style observations are noted inline — Cobra renders Example fields verbatim in the terminal, so ### will appear literally rather than as rendered headings.

Short: "Remove secrets from local keychain.",
Long: "Removes one or more named secrets from the local OS keychain.\nUse --all to remove every stored secret at once.",
Example: `# Remove a specific secret:
Example: `### Remove a specific secret:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LOW: Example section headers use ### (Markdown heading) instead of conventional # (shell comment)

Cobra does not render Markdown — the Example field is printed verbatim to the terminal. Replacing # Remove a specific secret: with ### Remove a specific secret: means users will see raw ### syntax in docker pass rm --help output.

The # prefix is both standard shell comment syntax and the de-facto Cobra/Docker CLI convention (used by Docker CLI, kubectl, and most Cobra-based tools). Consider reverting to # for consistent terminal UX.


const setExample = `
# Set a secret:
### Set a secret:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LOW: Example section headers use ### (Markdown heading) instead of conventional # (shell comment)

Same issue as rm.go: replacing # Set a secret: with ### Set a secret: causes docker pass set --help to display raw Markdown heading syntax in the terminal. Cobra prints Example strings as-is with no Markdown processing, so ### appears literally to users and deviates from the shell-comment convention.

Consider reverting to # for consistency with Docker CLI and Cobra ecosystem conventions.

@joe0BAB joe0BAB merged commit baa5e70 into main Apr 23, 2026
26 of 27 checks passed
@joe0BAB joe0BAB deleted the feat/docs branch April 23, 2026 14:55
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.

2 participants