Skip to content

[release-branch.go1.27] Skip doc reproducibility test if document is a copy - #2674

Merged
Davis Goodin (dagood) merged 1 commit into
microsoft/release-branch.go1.27from
dev/dagood/ok-gen-doc-copy-1.27
Oct 28, 2026
Merged

Davis Goodin (dagood) merged 1 commit into
microsoft/release-branch.go1.27from
dev/dagood/ok-gen-doc-copy-1.27

Conversation

@dagood

Copy link
Copy Markdown
Member

@dagood
Davis Goodin (dagood) requested a review from a team as a code owner October 28, 2026 18:56
Copilot AI balanced review requested due to automatic review settings October 28, 2026 18:56
@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.

🟢 Approval recommended

No active review issues remain for the documentation-copy detection and reproducibility-test updates.

No open issues 🎉

What changed in this PR

Ports #2672 to release-branch.go1.27, preventing documentation syncs from failing reproducibility tests when documents are maintained by automatic copying.

Changes:

  • Adds a shared copy-notice detector that handles LF and CRLF line endings.
  • Tests copied documents, ordinary content, and prefix edge cases.
  • Skips both crypto documentation reproducibility tests when the copy notice is present.
File Description
eng/​_util/​internal/​docutil/​docutil.go Adds detection of the automatic documentation copy notice.
eng/​_util/​internal/​docutil/​docutil_test.go Covers notice detection, line endings, and nonmatching content.
eng/​_util/​cmd/​updatecryptodocs/​updatecryptodocs_test.go Skips reproducibility checks for automatically copied documents.

Review effort: Balanced

@dagood
Davis Goodin (dagood) enabled auto-merge (squash) October 28, 2026 22:01
@dagood
Davis Goodin (dagood) merged commit 92f7c2d into microsoft/release-branch.go1.27 Oct 28, 2026
70 checks passed
@dagood
Davis Goodin (dagood) deleted the dev/dagood/ok-gen-doc-copy-1.27 branch October 28, 2026 22:35
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