Skip to content

docs: restore hardware key callout to top-level SSH signing tip#805

Merged
addisonbeck merged 2 commits into
mainfrom
update-commit-signing-docs
May 8, 2026
Merged

docs: restore hardware key callout to top-level SSH signing tip#805
addisonbeck merged 2 commits into
mainfrom
update-commit-signing-docs

Conversation

@addisonbeck
Copy link
Copy Markdown
Contributor

@addisonbeck addisonbeck commented May 5, 2026

🎟️ Tracking

https://bitwarden.slack.com/archives/GRN5AAZ6Y/p1778000211187909

📔 Objective

On #672 we moved the sk-backed key recommendation out of the main tip and into a nested step. Add it back at the top level so hardware keys remain visibly ranked as the highest-security option.

📸 Screenshots

@addisonbeck addisonbeck requested a review from a team as a code owner May 5, 2026 17:43
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 5, 2026

Deploying contributing-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 758ccf8
Status:⚡️  Build in progress...

View logs

On #672 we moved the sk-backed key recommendation out of the main tip and
into a nested step. Add it back at the top level so hardware keys remain
visibly ranked as the highest-security option.
@addisonbeck addisonbeck force-pushed the update-commit-signing-docs branch from 3c6e595 to a81a590 Compare May 5, 2026 17:47
Copy link
Copy Markdown
Contributor

@djsmith85 djsmith85 left a comment

Choose a reason for hiding this comment

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

FYI: Vince has some additional changes here: #806

@addisonbeck addisonbeck enabled auto-merge (squash) May 8, 2026 16:29
@addisonbeck addisonbeck merged commit 3652098 into main May 8, 2026
10 of 11 checks passed
@addisonbeck addisonbeck deleted the update-commit-signing-docs branch May 8, 2026 16:30
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.

3 participants