Update eng/doc based on 1.27 - #2466
Merged
Merged
Conversation
Davis Goodin (dagood)
commented
Aug 20, 2026
Davis Goodin (dagood)
commented
Member
- Typical updates removing unnecessary mentions of 1.25, particularly where that lets us simplify.
- Some reworking around cgo, cross-platform builds, and glibc. Centralize some information in a more reasonable place (IMO) in a super clear (I hope) way.
- Treat the cgo-less OpenSSL backend as something that users should expect to work for them, vs. something new. Tried to simplify the FIPS doc in particular with this viewpoint.
- Closes 2026-08-19 releases: 1.27.0-1 #2462
|
Azure Pipelines: Successfully started running 1 pipeline(s). There may be pipelines that require an authorized user to comment /azp run to run. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates engineering documentation to reflect Microsoft Go 1.27 behavior, especially around systemcrypto, cgo vs no-cgo OpenSSL backends, and simplifying/modernizing guidance by removing older 1.25-era details.
Changes:
- Updates the no-cgo (cgo-less) OpenSSL backend documentation and clarifies how it relates to the cgo OpenSSL backend.
- Simplifies the Migration Guide by removing an older (Go 1.25) variant of a
systemcrypto/cgo-related build error example. - Reworks FIPS documentation and adds a new cgo-focused section to cross-platform crypto docs to centralize guidance.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| eng/doc/NocgoOpenSSL.md | Clarifies terminology (“no-cgo” vs “cgo-less”) and selection behavior in 1.27+. |
| eng/doc/MigrationGuide.md | Removes Go 1.25-specific error example to reduce legacy guidance. |
| eng/doc/fips/README.md | Updates systemcrypto explanation and guidance for 1.27+ behavior; includes a small typo to fix. |
| eng/doc/CrossPlatformCryptography.md | Adds a new “Cgo” section and cgo availability table; however this file is generated and should not be edited directly. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: gdams <20224954+gdams@users.noreply.github.com>
George Adams (gdams)
approved these changes
Aug 21, 2026
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
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.