Skip to content

Update crypto docs to reflect current FreeBSD status - #2479

Merged
George Adams (gdams) merged 1 commit into
microsoft/mainfrom
dev/dagood/current-freebsd-doc
Aug 24, 2026
Merged

Update crypto docs to reflect current FreeBSD status#2479
George Adams (gdams) merged 1 commit into
microsoft/mainfrom
dev/dagood/current-freebsd-doc

Conversation

@dagood

Copy link
Copy Markdown
Member

Copilot AI lite review requested due to automatic review settings August 24, 2026 17:34
@dagood
Davis Goodin (dagood) requested a review from a team as a code owner August 24, 2026 17:34
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the generated cross-platform cryptography documentation and its generator header template to reflect FreeBSD systemcrypto support status across cgo-enabled vs cgo-disabled builds.

Changes:

  • Adjusts the FreeBSD row in the “Cgo enabled / Cgo disabled” support table.
  • Adds a new footnote describing expected FreeBSD cgo support changes, and propagates it into the generated doc output.

Reviewed changes

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

File Description
eng/doc/CrossPlatformCryptography.md Updates the generated documentation table/footnotes for FreeBSD cgo support status.
eng/_util/cmd/updatecryptodocs/header.md Updates the embedded header template that is used to generate CrossPlatformCryptography.md.
Suppressed comments (1)

eng/_util/cmd/updatecryptodocs/header.md:61

  • Footnote 3 says FreeBSD cgo support for all architectures is "expected in Go 1.27.1", but #2477 is already merged. If the docs are intended to reflect the current state of supported branches (per the PR description), this should be updated to reflect the current reality (or removed) rather than making a forward-looking statement that can become stale.
<sup>2</sup> Supported on **amd64** and **arm64** architectures.

<sup>3</sup> Support with cgo for all FreeBSD architectures is [expected in Go 1.27.1](https://github.com/microsoft/go/pull/2477).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread eng/_util/cmd/updatecryptodocs/header.md
@gdams
George Adams (gdams) enabled auto-merge (squash) August 24, 2026 18:22
@gdams
George Adams (gdams) merged commit 1ac5c69 into microsoft/main Aug 24, 2026
59 checks passed
@gdams
George Adams (gdams) deleted the dev/dagood/current-freebsd-doc branch August 24, 2026 18:25
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