Skip to content

Update docs-verifier action and add env variables - #3436

Merged
gewarren merged 3 commits into
mainfrom
gewarren-patch-1-1
Jul 30, 2026
Merged

Update docs-verifier action and add env variables#3436
gewarren merged 3 commits into
mainfrom
gewarren-patch-1-1

Conversation

@gewarren

Copy link
Copy Markdown
Contributor

Updated the docs verifier action to a new version and added environment variables for GitHub PR number and token.

Updated the docs verifier action to a new version and added environment variables for GitHub PR number and token.
Copilot AI review requested due to automatic review settings July 29, 2026 17:39
@gewarren
gewarren requested a review from a team as a code owner July 29, 2026 17:39
@gewarren
gewarren enabled auto-merge (squash) July 29, 2026 17:39
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 0a44ca7:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/docs-verifier.yml ✅Succeeded

For more details, please refer to the build report.

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 GitHub Actions workflow used to validate PR builds by switching to a newer docs-verifier action and centralizing the PR metadata/token environment variables.

Changes:

  • Switched the Validate step to use dotnet/docs-tools/actions/docs-verifier pinned to a new commit SHA.
  • Moved GITHUB_PR_NUMBER and GITHUB_TOKEN environment variables to workflow scope.

Comment thread .github/workflows/docs-verifier.yml Outdated
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

gewarren added 2 commits July 29, 2026 15:57
Moved environment variables into the Validate step for better scoping.
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit ce14921:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/docs-verifier.yml ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit d6c4ea4:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/docs-verifier.yml ✅Succeeded

For more details, please refer to the build report.

@wadepickett wadepickett 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.

@gewarren: Approved.

@gewarren
gewarren merged commit f2bee93 into main Jul 30, 2026
8 checks passed
@gewarren
gewarren deleted the gewarren-patch-1-1 branch July 30, 2026 17: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